Closed
Bug 911946
Opened 11 years ago
Closed 11 years ago
gaia-ui-test & gaia-unit being scheduled twice on Try runs
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: catlee)
References
Details
(Whiteboard: [capacity][b2g])
Attachments
(1 file, 1 obsolete file)
1.19 KB,
patch
|
rail
:
review+
catlee
:
checked-in+
|
Details | Diff | Splinter Review |
We have what appears to be an additional duplicate job scheduled on try runs:
https://tbpl.mozilla.org/?tree=Try&jobname=gaia
Reporter | ||
Updated•11 years ago
|
Summary: Gaia unit tests being scheduled twice on Try runs → gaia-ui-test being scheduled twice on Try runs
Reporter | ||
Comment 1•11 years ago
|
||
Both are in fact (need showall=1)
Summary: gaia-ui-test being scheduled twice on Try runs → gaia-ui-test & gaia-unit being scheduled twice on Try runs
Reporter | ||
Updated•11 years ago
|
Whiteboard: [capacity]
Assignee | ||
Comment 2•11 years ago
|
||
weird!
so there are two sendchanges in this log: https://tbpl.mozilla.org/php/getParsedLog.php?id=27642850&tree=Try&full=1 tracking down where these are coming from
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → catlee
Whiteboard: [capacity] → [capacity][b2g]
Assignee | ||
Comment 3•11 years ago
|
||
Attachment #802415 -
Flags: review?(bhearsum)
Updated•11 years ago
|
Attachment #802415 -
Flags: review?(bhearsum) → review+
Assignee | ||
Updated•11 years ago
|
Attachment #802415 -
Flags: checked-in+
Comment 4•11 years ago
|
||
in production
Comment 5•11 years ago
|
||
Comment 6•11 years ago
|
||
See bug 914821 for bustage details.
Assignee | ||
Updated•11 years ago
|
Attachment #802415 -
Flags: checked-in+ → checked-in-
Assignee | ||
Comment 7•11 years ago
|
||
turns out that "if not uploadMulti" is very very different from "if uploadMulti".
this also fixes uploading the multilocale builds instead of uploading the en-US builds again.
Attachment #802415 -
Attachment is obsolete: true
Attachment #803699 -
Flags: review?(rail)
Comment 8•11 years ago
|
||
Comment on attachment 803699 [details] [diff] [review]
don't sendchange twice
Makes sense!
Attachment #803699 -
Flags: review?(rail) → review+
Assignee | ||
Updated•11 years ago
|
Attachment #803699 -
Flags: checked-in+
Assignee | ||
Comment 9•11 years ago
|
||
This got deployed at some point.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
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
•