Closed
Bug 1352367
Opened 8 years ago
Closed 8 years ago
[push-apk] taskgraph: Deactivate dry-run on aurora and beta
Categories
(Release Engineering :: Release Automation: Other, defect, P1)
Release Engineering
Release Automation: Other
Tracking
(firefox55 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: jlorenzo, Assigned: jlorenzo)
References
Details
Attachments
(2 files)
(deleted),
text/x-review-board-request
|
mozilla
:
review+
|
Details |
(deleted),
patch
|
Details | Diff | Splinter Review |
Bug 1317783 is a bit overloaded with attachments, It's better to start off a new bug.
Bug 1317783 landed with the dry-run option on[1]. It's now time to activate that projects which are known to work.
[1] https://tools.taskcluster.net/task-group-inspector/#/JF1eWHTJRnGwuiiWQDuVJg/VL4G5ckHQfuraiQ0MyZmIg?_k=aqnv50
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•8 years ago
|
||
On aurora, I let run 0[1] to be a dry run. Then I temporarily modified pushapkscript to forget about the dry-run option in the payload[2], which resulted to run 1[3]. As run 1 passed, we can now deactivate dry-run on aurora.
[1] https://tools.taskcluster.net/task-inspector/#VL4G5ckHQfuraiQ0MyZmIg/0
[2] https://github.com/mozilla-releng/pushapkscript/blob/c4cdaf0f173dcda25fa19f6aab6806d1c016a4c0/pushapkscript/googleplay.py#L30
[3] https://tools.taskcluster.net/task-inspector/#VL4G5ckHQfuraiQ0MyZmIg/0
Assignee: nobody → jlorenzo
Updated•8 years ago
|
Priority: -- → P1
Comment 3•8 years ago
|
||
mozreview-review |
Comment on attachment 8853376 [details]
Bug 1352367 - [push-apk] deactivate dry-run option on aurora and beta
https://reviewboard.mozilla.org/r/125492/#review128136
Attachment #8853376 -
Flags: review?(aki) → review+
Assignee | ||
Comment 4•8 years ago
|
||
Landed on aurora at: https://hg.mozilla.org/releases/mozilla-aurora/rev/bd329aa7dbcd8a9c3ee467d393b3c606f347419f
This cannot make central yet because of bug 1352477. If I didn't land that patch on mozilla-aurora, no aurora builds would have happened over the weekend. I'll keep an eye out tomorrow to make sure it didn't break anything.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 6•8 years ago
|
||
Similarly to comment 2, I just uploaded 53.0b9 at [1]. We can now deactivate dry-run on beta as well.
[1] https://tools.taskcluster.net/task-group-inspector/#/ehxz2GL0RU2IpOy5aOgN6Q/GNUsvLvzTa2e-GXxMm0pHQ?_k=u6p2rz
Assignee | ||
Updated•8 years ago
|
Keywords: leave-open
Pushed by jlorenzo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/00f026055338
[push-apk] deactivate dry-run option on aurora and beta r=aki
Assignee | ||
Comment 8•8 years ago
|
||
Uplift of attachment 8853376 [details] (because one part landed in aurora in comment 4)
Assignee | ||
Comment 9•8 years ago
|
||
Assignee | ||
Comment 10•8 years ago
|
||
Attachment 8853376 [details] landed on beta at https://hg.mozilla.org/releases/mozilla-beta/rev/acc4c147ae006e22c51820caf6386054d6e05a31. Next step is to handle release. However, this will happen in about 2 weeks. In order to not space the different patches too much, I prefer to open a follow up bug (bug 1354038).
Hence, this bug can be closed once the patch reaches central.
Keywords: leave-open
Summary: [push-apk] taskgraph: Deactivate dry-run → [push-apk] taskgraph: Deactivate dry-run on aurora and beta
Comment 11•8 years ago
|
||
bugherder |
You need to log in
before you can comment on or make changes to this bug.
Description
•