766030 - verify.py should run instead of updateSUT/cleanup.py for tegra jobs

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Justin Wood (:Callek)]

Attachment: [tools] v1

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.

Comment 1

[Justin Wood (:Callek)]

Attachment: [buildbotcustom] v0.5

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.

Comment 2

[Justin Wood (:Callek)]

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.

Comment 3

[Justin Wood (:Callek)]

http://hg.mozilla.org/build/tools/rev/05bbab1ef094

Comment 4

[Justin Wood (:Callek)]

tools side is deployed now.

Comment 5

[Justin Wood (:Callek)]

Pushed buildbotcustom patch to default, merge at will.

http://hg.mozilla.org/build/buildbotcustom/rev/f5385d109228

Comment 6

[Armen [:armenzg]]

This is now live in production.