gaia-ui-test & gaia-unit being scheduled twice on Try runs

RESOLVED FIXED

Status

RESOLVED FIXED
5 years ago
4 months ago

People

(Reporter: emorley, Assigned: catlee)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [capacity][b2g])

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

5 years ago
We have what appears to be an additional duplicate job scheduled on try runs:
https://tbpl.mozilla.org/?tree=Try&jobname=gaia
(Reporter)

Updated

5 years ago
Summary: Gaia unit tests being scheduled twice on Try runs → gaia-ui-test being scheduled twice on Try runs
(Reporter)

Comment 1

5 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

5 years ago
Whiteboard: [capacity]
(Assignee)

Comment 2

5 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

5 years ago
Assignee: nobody → catlee
Whiteboard: [capacity] → [capacity][b2g]
(Assignee)

Comment 3

5 years ago
Created attachment 802415 [details] [diff] [review]
don't sendchange twice
Attachment #802415 - Flags: review?(bhearsum)
Attachment #802415 - Flags: review?(bhearsum) → review+
(Assignee)

Updated

5 years ago
Attachment #802415 - Flags: checked-in+
in production
See bug 914821 for bustage details.
(Assignee)

Updated

5 years ago
Attachment #802415 - Flags: checked-in+ → checked-in-
(Assignee)

Comment 7

5 years ago
Created attachment 803699 [details] [diff] [review]
don't sendchange twice

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 on attachment 803699 [details] [diff] [review]
don't sendchange twice

Makes sense!
Attachment #803699 - Flags: review?(rail) → review+
(Assignee)

Updated

5 years ago
Attachment #803699 - Flags: checked-in+
(Assignee)

Comment 9

5 years ago
This got deployed at some point.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Component: General Automation → General
Product: Release Engineering → Release Engineering
You need to log in before you can comment on or make changes to this bug.