819482 - Remove retry code from B2G automation once bug 816216 propagates

Status: RESOLVED FIXED

(Testing :: General, defect)

Description

[Andrew Halberstadt [:ahal]]

We retry adb push at several points in B2G automation since adb sometimes freezes. Bug 816216 will fix the problem within devicemanager so these will no longer be needed:
http://mxr.mozilla.org/mozilla-central/search?string=if+retry+%3D%3D+push_attempts&find=&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

Comment 1

[Mihnea Dobrescu-Balaur (:mihneadb)]

Attachment: refactor b2gautomation according to new retryLimit param

Comment 2

[Andrew Halberstadt [:ahal]]

Comment on attachment 689843 [details] [diff] [review]
refactor b2gautomation according to new retryLimit param

Review of attachment 689843 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, but we can't land this until the other bug lands in github and gets merged to m-c.

Comment 3

[Mihnea Dobrescu-Balaur (:mihneadb)]

No rush, just wanted to be done with it

Comment 4

[Andrew Halberstadt [:ahal]]

Pushed to inbound minus the hunk in install_gecko() since that was re-written and is about to be re-written again (I get rid of the retry code in a patch I'm about to land).

https://hg.mozilla.org/integration/mozilla-inbound/rev/e75caf89e1f9

Comment 5

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/e75caf89e1f9