Closed
Bug 1212887
Opened 9 years ago
Closed 9 years ago
10.10.5 tests run on try when 10.10 is explicitly specified
Categories
(Infrastructure & Operations Graveyard :: CIDuty, task)
Infrastructure & Operations Graveyard
CIDuty
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kmoir, Assigned: kmoir)
References
Details
Attachments
(2 files, 1 obsolete file)
|
938 bytes,
patch
|
Callek
:
review+
kmoir
:
checked-in+
|
Details | Diff | Splinter Review |
|
1.02 KB,
patch
|
kmoir
:
checked-in+
|
Details | Diff | Splinter Review |
10.10.5 tests should not run by on try unless 10.10.5 is specified but this seems to be happening. This is not good because they are not greened up yet see bug 1210395 https://hg.mozilla.org/try/rev/33bc0551337f
|
Assignee | |
Comment 1•9 years ago
|
||
Callek_cloud kmoir: #2 (10.10 running 10.10.5) is dicey unless we rename 10.10 to 10.10.0 aiui: https://dxr.mozilla.org/build-central/source/buildbotcustom/try_parser.py#191 Callek_cloud since that code means 10.10 does indeed match 10.10.5, but is setup in a way that makes this "harder" to fix
|
|
Assignee | |
Comment 2•9 years ago
|
||
Alin suggested using exclusions to get around this temporarily which works https://treeherder.mozilla.org/#/jobs?repo=try&revision=96312aa64044 not sure if this patch will do the trick
|
|
Assignee | |
Comment 3•9 years ago
|
||
Attachment #8671996 -
Attachment is obsolete: true
|
|
Assignee | |
Updated•9 years ago
|
Attachment #8672000 -
Flags: review?(bugspam.Callek)
|
||
Comment 4•9 years ago
|
||
Comment on attachment 8672000 [details] [diff] [review] bug1212887.patch Review of attachment 8672000 [details] [diff] [review]: ----------------------------------------------------------------- slightly concerned that some humans and other cli tools won't let this change solve this well. But I'm happy giving this a shot.
Attachment #8672000 -
Flags: review?(bugspam.Callek) → review+
|
|
Assignee | |
Comment 5•9 years ago
|
||
Comment on attachment 8672000 [details] [diff] [review] bug1212887.patch and deployed to trychooser which looks good https://wiki.mozilla.org/ReleaseEngineering/How_To/Update_a_Releng_Cluster_Site
Attachment #8672000 -
Flags: checked-in+
|
|
Assignee | |
Comment 6•9 years ago
|
||
Thanks Alin for this idea, it worked. I'll leave this open because we'll want to revert it when the 10.10.5 tests are green
|
||
Comment 7•9 years ago
|
||
ok, 10.10.5 is live- shall we revert things?
|
|
Assignee | |
Comment 8•9 years ago
|
||
patch to revert
|
|
Assignee | |
Updated•9 years ago
|
Attachment #8689520 -
Flags: checked-in+
|
|
Assignee | |
Comment 9•9 years ago
|
||
new version of trychooser deployed
Assignee: nobody → kmoir
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
|
||
Updated•6 years ago
|
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
|
||
Updated•4 years ago
|
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•