Closed
Bug 837356
Opened 12 years ago
Closed 12 years ago
sut_lib soft_reboot_and_verify sometimes prints out "Remote Device Error:" even in success conditions.
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Callek, Assigned: Callek)
References
Details
Attachments
(1 file)
1.06 KB,
patch
|
Callek
:
review+
|
Details | Diff | Splinter Review |
I suspect this is a large part of some of the sheriff issue's we've been seeing lately [todo tie to bugs]
Especially on the pandas.
So what happens is if waitForDevice fails 3 times in a row, we print the Remote Device Error, and then pass back into the softreboot loop, which tries 5 seperate times as well to reboot+wait.
If it succeeds the second time (which is very common) all is well for THIS script, but buildbot sees "Remote Device Error" in the log and throws its hands up, "ooo infra error, ok lets fail out"
When the actual buildbot "steps" didn't return failure either.
I'm not sure atm what the best fix here is, but a fix does need to be made.
Updated•12 years ago
|
OS: Windows 7 → Android
Hardware: x86_64 → ARM
Comment 1•12 years ago
|
||
Attachment #710344 -
Flags: review?(bugspam.Callek)
Assignee | ||
Comment 2•12 years ago
|
||
Comment on attachment 710344 [details] [diff] [review]
don't print out
I was hoping for a more in depth audit/solution, but forgot I was buildduty this week so I can't invest as much (if any) time as I'd like. Lets go with this pending a possible better choice, I'll deploy it tomorrow
Attachment #710344 -
Flags: review?(bugspam.Callek) → review+
Comment 3•12 years ago
|
||
landed in the build/tools repository, feel free to make this a leave open bug for a larger scale cleanup.
Assignee | ||
Comment 4•12 years ago
|
||
Deployed just now.
For tracking sanity I'm resolving this bug, and we can do new bugs once we get further along in cleanups.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•