600267 - add reboot event to output stream from SUTAgentAndroid

Status: RESOLVED DUPLICATE of bug 600266

(Testing :: General, defect)

Description

[Mike Taylor [:bear]]

I would like to suggest adding a reboot notification to the same data channel that outputs the heartbeat text.

This will allow downstream consumers to know that a reboot event has happened and trigger any behaviour instead of trying to overload existing "gone away" behaviour using the heartbeat.

Comment 1

[Mike Taylor [:bear]]

marking this as a blocker.

I've been trying to work with various "sleep" methods during the reboot but I can't get around the fact that the slave proxy's socket to the heartbeat stream is dropped and that triggers a restarting of the slave.

Having the SUT Agent report a reboot would let the slave proxy be proactive in searching for a new heartbeat stream.

Comment 2

[Mike Taylor [:bear]]

I tested the binary that bmoss gave me and the new output for rebooting works like a charm.  The buildbot slave is now able to know exactly when it happens without any messy sleep calls.

Comment 3

[Robert Sayre]

Is there a patch available for this?

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

the patch ended up here: https://bugzilla.mozilla.org/show_bug.cgi?id=600266.

Comment 5

[Robert Sayre]

so, what is this bug about?

Comment 6

[Joel Maher ( :jmaher ) (UTC -8)]

the fix was done at the same time in the same patch.  I recommend we duplicate this bug as dup of bug 600266.  clint or bmoss?

Comment 7

[Mike Taylor [:bear]]

this bug was made invalid by the requested item landing in another bug's patch. 

closing