Closed
Bug 766030
Opened 13 years ago
Closed 13 years ago
verify.py should run instead of updateSUT/cleanup.py for tegra jobs
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Callek, Assigned: Callek)
Details
Attachments
(2 files)
924 bytes,
patch
|
bear
:
review+
|
Details | Diff | Splinter Review |
5.30 KB,
patch
|
bear
:
review+
|
Details | Diff | Splinter Review |
Ok, we should start running verify.py instead of updateSUT/cleanup in here. It will properly set the error.flg and alert us in a good way if stuff goes wonky, instead of letting us fail in a bad way with cleanup.
We currently run cleanup two places, setup and teardown, but this patch will only do verify during setup, and let cleanup.py continue to run after setup, this seems safest until I get some of the flags into verify that make sense to keep us from spending tons of time in here at the end of every job.
(Yes this will mean we do verify actions more than once in some cases, but I feel this is worth it).
First the prelim patch, allow verify.py to get its tegra info from the environment.
Attachment #634306 -
Flags: review?(bear)
Assignee | ||
Comment 1•13 years ago
|
||
This patch has not even been run through checkconfig, let alone a dump_master or staging, I will do all of the above before I land though.
Attachment #634309 -
Flags: review?(bear)
Assignee | ||
Comment 2•13 years ago
|
||
Comment on attachment 634309 [details] [diff] [review]
[buildbotcustom] v0.5
O too be clear, without redoing how steps are ordered/defined in Talos Factory, I'm stuck doing the verify the same way before and after a job right now.
Changing that is a bit more than I feel we should/need to tackle in this bug/quarter though, so we can address later, if that is ok.
Updated•13 years ago
|
Attachment #634309 -
Flags: review?(bear) → review+
Updated•13 years ago
|
Attachment #634306 -
Flags: review?(bear) → review+
Assignee | ||
Comment 3•13 years ago
|
||
Assignee | ||
Comment 4•13 years ago
|
||
tools side is deployed now.
Assignee | ||
Comment 5•13 years ago
|
||
Pushed buildbotcustom patch to default, merge at will.
http://hg.mozilla.org/build/buildbotcustom/rev/f5385d109228
Comment 6•13 years ago
|
||
This is now live in production.
Assignee | ||
Updated•13 years ago
|
Status: NEW → RESOLVED
Closed: 13 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
•