Closed
Bug 1232571
Opened 9 years ago
Closed 9 years ago
Enable emu-x86-kk marionette-webapi tests on m-c and integration branches
Categories
(Firefox Build System :: Task Configuration, task)
Firefox Build System
Task Configuration
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: edgar, Assigned: edgar)
References
Details
(Whiteboard: [EMU] [CI])
Attachments
(2 files, 1 obsolete file)
Now marionette-webapi is green on try now [1], it's time to enable it on m-c and integration branches.
[1] https://treeherder.mozilla.org/#/jobs?repo=try&revision=3ccfa8e00979&group_state=expanded
Assignee | ||
Comment 1•9 years ago
|
||
Bug 1232571 - Enable emu-x86-kk marionette-webapi tests on m-c and integration branches
Assignee | ||
Comment 2•9 years ago
|
||
Assignee | ||
Comment 3•9 years ago
|
||
Review request updated; see interdiff: https://reviewboard.mozilla.org/r/27943/diff/1-2/
Attachment #8698769 -
Flags: review?(garndt)
Assignee | ||
Updated•9 years ago
|
Attachment #8698321 -
Attachment is obsolete: true
Comment 4•9 years ago
|
||
Comment on attachment 8698769 [details]
MozReview Request: Bug 1232571 - Enable emu-x86-kk marionette-webapi tests on m-c and integration branches
https://reviewboard.mozilla.org/r/27943/#review25179
Attachment #8698769 -
Flags: review?(garndt) → review+
Assignee | ||
Comment 5•9 years ago
|
||
Comment on attachment 8698321 [details]
MozReview Request: Bug 1232571 - Enable emu-x86-kk marionette-webapi tests on m-c and integration branches. r=garndt
Review request updated; see interdiff: https://reviewboard.mozilla.org/r/27943/diff/1-2/
Attachment #8698321 -
Attachment description: MozReview Request: Bug 1232571 - Enable emu-x86-kk marionette-webapi tests on m-c and integration branches → MozReview Request: Bug 1232571 - Enable emu-x86-kk marionette-webapi tests on m-c and integration branches. r=garndt
Attachment #8698321 -
Attachment is obsolete: false
Attachment #8698321 -
Flags: review?(garndt)
Assignee | ||
Comment 6•9 years ago
|
||
Comment on attachment 8698321 [details]
MozReview Request: Bug 1232571 - Enable emu-x86-kk marionette-webapi tests on m-c and integration branches. r=garndt
Just rebase.
Attachment #8698321 -
Flags: review?(garndt) → review+
Assignee | ||
Updated•9 years ago
|
Attachment #8698769 -
Attachment is obsolete: true
Assignee | ||
Comment 8•9 years ago
|
||
Before enabling on production, I would like to monitor the status on staging for a while first.
Keywords: leave-open
Comment 9•9 years ago
|
||
bugherder |
Assignee | ||
Comment 10•9 years ago
|
||
(In reply to Edgar Chen [:edgar][:echen] from comment #8)
> Before enabling on production, I would like to monitor the status on staging
> for a while first.
Status of marionette-webapi looks good on staging:
https://treeherder.allizom.org/#/jobs?repo=b2g-inbound&filter-searchStr=mnw
It's time to enable it on production.
Assignee | ||
Comment 11•9 years ago
|
||
Assignee | ||
Comment 12•9 years ago
|
||
Comment on attachment 8700487 [details] [diff] [review]
Enable marionette-webapi on production, v1
Review of attachment 8700487 [details] [diff] [review]:
-----------------------------------------------------------------
Somehow, I don't know why, ReviewBoard didn't allow me to upload patch for this bug. So I send r? without using ReviewBoard.
Attachment #8700487 -
Flags: review?(garndt)
Comment 13•9 years ago
|
||
Comment on attachment 8700487 [details] [diff] [review]
Enable marionette-webapi on production, v1
So if these are getting enabled on production, we need to know who owns these tests in case of failures as these will be tier 1 jobs and can possibly cause tree closures and backouts.
The change itself looks ok.
Attachment #8700487 -
Flags: review?(garndt) → review+
Updated•9 years ago
|
Whiteboard: [EMU] [CI]
Assignee | ||
Comment 14•9 years ago
|
||
(In reply to Greg Arndt [:garndt] from comment #13)
> Comment on attachment 8700487 [details] [diff] [review]
> Enable marionette-webapi on production, v1
>
> So if these are getting enabled on production, we need to know who owns
> these tests in case of failures as these will be tier 1 jobs and can
> possibly cause tree closures and backouts.
>
> The change itself looks ok.
Ken Chang would be the owner. Thank you.
Assignee | ||
Updated•9 years ago
|
Keywords: leave-open
Comment 16•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 17•9 years ago
|
||
(In reply to Carsten Book [:Tomcat] from comment #16)
> https://hg.mozilla.org/mozilla-central/rev/60a85952d9fd
What amazing news! Thanks Edgar! Thanks teams :)
Updated•7 years ago
|
Product: TaskCluster → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•