811116 - Deploy updated android and maintenance BMM scripts

Status: RESOLVED FIXED

(Infrastructure & Operations :: RelOps: General, task)

Description

[Dustin J. Mitchell [:dustin] (he/him)]

Finally, I integrated the device-side shell scripts with the server-side Lifeguard state machine.  These have successfully imaged a panda (well, one out of three, but they'll try about 200 times).

Comment 1

[Dustin J. Mitchell [:dustin] (he/him)]

Attachment: bug811116.patch

Comment 2

[Jake Watkins [:dividehex]]

Comment on attachment 680816 [details] [diff] [review]
bug811116.patch

>+
>+WGET_OUTPUT=$(wget -nv --directory-prefix=/opt/artifacts/ http://${IMAGING_SERVER}/artifacts/panda-jb-gcc47-tilt-stable-blob-12.08-release/system.tar.bz2 2>&1)
>+log "${WGET_OUTPUT}"
>+WGET_OUTPUT=$(wget -nv --directory-prefix=/opt/artifacts/ http://${IMAGING_SERVER}/artifacts/panda-jb-gcc47-tilt-stable-blob-12.08-release/userdata.tar.bz2 2>&1)
>+log "${WGET_OUTPUT}"
>+WGET_OUTPUT=$(wget -nv --directory-prefix=/opt/artifacts/ http://${IMAGING_SERVER}/artifacts/panda-jb-gcc47-tilt-stable-blob-12.08-release/boot.tar.bz2  2>&1)
>+log "${WGET_OUTPUT}"
>+

These artifacts are not the current ones for production and should only be used for testing


>+log "Extracting artifacts"
>+tar -jxf /opt/artifacts/boot.tar.bz2 -C /opt/mnt/boot --strip=1 boot/uImage boot/uInitrd

fat doesn't like it when tar tries to set owner.  it may work but it will exit tar with errors.  add the --no-same-owner flag
eg.
tar -jx --no-same-owner -f /opt/artifacts/boot.tar.bz2 -C /opt/mnt/boot --strip=1 boot/uImage boot/uInitrd


>+log "Unmounting partitions"
>+umount /opt/mnt/boot /opt/mnt/system /opt/mnt/userdata

call sync before umounting here otherwise umount can sometimes fail


>+# signal an event to our lifeguard server, retrying as necessary
>+lifeguard_event() {
>+    local retry
>+    local event="$1"
>+    local url="http://${IMAGING_SERVER}/api/device/${HOSTNAME}/event/${event}/"
>+
>+    # retry forever, assuming lifeguard will kill us if this doesn't work
>+    while true; do
>+        log "Submitting lifeguard event at $url"
>+        wget -O/dev/null $url && return 0
>+        sleep 60
>+    done
>+}

According to the API this is call should be HTTP POST
Use --post-data='' with wget or correct API docs



>+# get and source utils
>+wget --post-data='' http://<%= @ipaddress %>/scripts/liveutil.sh || exit 1
>+source liveutil.sh

no need for HTTP POST here

Comment 3

[Dustin J. Mitchell [:dustin] (he/him)]

(In reply to Jake Watkins [:dividehex] from comment #2)
> According to the API this is call should be HTTP POST
> Use --post-data='' with wget or correct API docs

Actually, GET is legal:
  http://hg.mozilla.org/build/bmm/file/tip/API.txt#l97

The rest I'll fix.  Thanks!

Comment 4

[Dustin J. Mitchell [:dustin] (he/him)]

Attachment: bug811116-r2.patch

This seems to work as far as puppet is concerned, but the pandas aren't behaving well enough for me to actually test it.

Comment 5

[Dustin J. Mitchell [:dustin] (he/him)]

I was being stupid.  This basically works, so review away.

Comment 6

[Jake Watkins [:dividehex]]

Comment on attachment 681047 [details] [diff] [review]
bug811116-r2.patch

lgtm

Comment 7

[Dustin J. Mitchell [:dustin] (he/him)]

landed