Merge lp://qastaging/~javier.collado/utah/bug1197881 into lp://qastaging/utah

Proposed by Javier Collado
Status: Merged
Merged at revision: 966
Proposed branch: lp://qastaging/~javier.collado/utah/bug1197881
Merge into: lp://qastaging/utah
Diff against target: 72 lines (+18/-8)
3 files modified
debian/changelog (+3/-0)
examples/run_utah_phablet.py (+9/-5)
utah/provisioning/touch.py (+6/-3)
To merge this branch: bzr merge lp://qastaging/~javier.collado/utah/bug1197881
Reviewer Review Type Date Requested Status
Andy Doan (community) Approve
Review via email: mp+173999@code.qastaging.launchpad.net

Description of the change

This branch applies a retry strategy to workaround the failures when running
`adb shell getprop` command.

An example of the retry strategy working can be seen here (look for
"retrying"):
https://jenkins.qa.ubuntu.com/job/memevent-saucy-touch-armhf-default-mako-01/56/consoleFull

To post a comment you must log in.
Revision history for this message
Javier Collado (javier.collado) wrote :

Note that `ADBShellError` is now a subclass of `UTAHException` because that's
required by the `retry` function to work correctly.

Revision history for this message
Max Brustkern (nuclearbob) wrote :

This looks good to me.

Revision history for this message
Andy Doan (doanac) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches