Closed
Bug 888118
Opened 12 years ago
Closed 12 years ago
Mozharness desktop unittests don't use the in tree mozharness config
Categories
(Release Engineering :: Applications: MozharnessCore, defect)
Release Engineering
Applications: MozharnessCore
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ahal, Assigned: ahal)
References
Details
(Whiteboard: [mozharness])
Attachments
(1 file, 1 obsolete file)
4.46 KB,
patch
|
mozilla
:
review+
|
Details | Diff | Splinter Review |
This is because they selectively extract tests.zip and therefore the config directory doesn't exist. See https://hg.mozilla.org/build/mozharness/file/27b230a75983/scripts/desktop_unittest.py#l270
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #768689 -
Flags: review?(aki)
Comment 2•12 years ago
|
||
Comment on attachment 768689 [details] [diff] [review]
Patch 1.0 - also extract the config directory
Does this config/ directory exist on all trees? (This patch will affect all trees except maybe esr17, and will most likely cause burning without the config/ dir.)
Attachment #768689 -
Flags: review?(aki) → review+
Assignee | ||
Comment 3•12 years ago
|
||
Oh good call. I just checked, it's on b2g18* but not release (or aurora or beta probably). I guess my options are to either land the tree_config patch everywhere including release, or just make desktop mochitests extract the whole zip file :(
Comment 4•12 years ago
|
||
fwiw I'm totally fine having to push this manually to try to let bug 872164 ride the trains normally
Assignee | ||
Comment 5•12 years ago
|
||
(In reply to :Felipe Gomes from comment #4)
> fwiw I'm totally fine having to push this manually to try to let bug 872164
> ride the trains normally
Nah, this patch is against mozharness, so no way to push it to try. The problem is that all test jobs across all branches use the same mozharness scripts, which ironically is the reason we wanted in-tree mozharness scripts in the first place
Assignee | ||
Comment 6•12 years ago
|
||
So I did some digging around and it looks like when a match like this isn't found the return code is always 11. See http://www.info-zip.org/mans/unzip.html#DIAGNOSTICS
I verified this locally, but also pushed a patch to ash-mozharness to make sure it works on all platforms:
http://hg.mozilla.org/users/asasaki_mozilla.com/ash-mozharness/rev/a0e7aecd9241
Waiting for results here:
https://tbpl.mozilla.org/?tree=Ash&rev=10dcc800e67d
Assignee | ||
Comment 7•12 years ago
|
||
So you're right that the old patch would have caused bustage where "config" didn't exist. By allowing a return code of 11 though, this seems to work. I tested by adding a junk folder to the extract list on mozharness ash: http://hg.mozilla.org/users/asasaki_mozilla.com/ash-mozharness/rev/7cb05e1d1695
Attachment #768689 -
Attachment is obsolete: true
Attachment #770943 -
Flags: review?(aki)
Comment 8•12 years ago
|
||
Comment on attachment 770943 [details] [diff] [review]
Patch 2.0 - extract config and allow ret code 11 from tests unzip
Cool, good to see success_codes isn't a one trick pony.
Attachment #770943 -
Flags: review?(aki) → review+
Assignee | ||
Comment 9•12 years ago
|
||
https://hg.mozilla.org/build/mozharness/rev/703427b7e1ee
Still needs to be merged to production before live.
Comment 10•12 years ago
|
||
in production
Assignee | ||
Comment 11•12 years ago
|
||
Felipe, your previous patch should work now. I have a try run here to test it out:
https://tbpl.mozilla.org/?tree=Try&rev=f1e83b72d873
(if you see --console-level=DEBUG in the arguments that means it's working).
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 12•12 years ago
|
||
Thanks Andrew! I had also sent another try run earlier today and verified it works!
https://tbpl.mozilla.org/?tree=Try&rev=5d41b5109c3c
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
Updated•11 years ago
|
Component: General Automation → Mozharness
You need to log in
before you can comment on or make changes to this bug.
Description
•