Closed
Bug 1425985
Opened 7 years ago
Closed 7 years ago
Error out when a task exceeds MAX_DEPENDENCIES.
Categories
(Release Engineering :: Release Automation: Other, enhancement)
Release Engineering
Release Automation: Other
Tracking
(firefox59 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox59 | --- | fixed |
People
(Reporter: mozilla, Assigned: mozilla)
References
Details
Attachments
(4 files)
No description provided.
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Attachment #8937567 -
Flags: review?(aki) → review+
Assignee | ||
Comment 4•7 years ago
|
||
mozreview-review |
Comment on attachment 8937567 [details]
bug 1425985 - Add check for max dependencies.
https://reviewboard.mozilla.org/r/208246/#review214008
Assignee | ||
Comment 5•7 years ago
|
||
We should be able to autoland these patches once they're all r+'ed.
Assignee | ||
Comment 6•7 years ago
|
||
Comment hidden (mozreview-request) |
Assignee | ||
Comment 8•7 years ago
|
||
Dustin, Callek - just a heads up about these changes. They're largely relpro, other than MAX_DEPENDENCIES and the associated test in task.py.
Assignee | ||
Comment 9•7 years ago
|
||
(In reply to Aki Sasaki [:aki] from comment #8)
> Dustin, Callek - just a heads up about these changes. They're largely
> relpro, other than MAX_DEPENDENCIES and the associated test in task.py.
(Catlee's patch; I pushed it to mozreview and r+'ed it.)
Comment 10•7 years ago
|
||
mozreview-review |
Comment on attachment 8937567 [details]
bug 1425985 - Add check for max dependencies.
https://reviewboard.mozilla.org/r/208246/#review214034
Attachment #8937567 -
Flags: review+
Comment 11•7 years ago
|
||
mozreview-review |
Comment on attachment 8937627 [details]
bug 1425985 - fix fennec post-beetmover-dummy.
https://reviewboard.mozilla.org/r/208300/#review214150
Attachment #8937627 -
Flags: review?(bhearsum) → review+
Comment 12•7 years ago
|
||
mozreview-review |
Comment on attachment 8937569 [details]
bug 1425985 - fix beetmover-cdns deps.
https://reviewboard.mozilla.org/r/208250/#review214152
Attachment #8937569 -
Flags: review?(bhearsum) → review+
Comment 13•7 years ago
|
||
mozreview-review |
Comment on attachment 8937568 [details]
bug 1425985 - set MAX_DEPENDENCIES to 99.
https://reviewboard.mozilla.org/r/208248/#review214154
Attachment #8937568 -
Flags: review?(bhearsum) → review+
Comment 14•7 years ago
|
||
Pushed by bhearsum@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c287bb73abc5
Add check for max dependencies. r=dustin
https://hg.mozilla.org/integration/autoland/rev/486a68bfad7a
set MAX_DEPENDENCIES to 99. r=bhearsum
https://hg.mozilla.org/integration/autoland/rev/89f56678fd20
fix beetmover-cdns deps. r=bhearsum
https://hg.mozilla.org/integration/autoland/rev/7cc4eeacaaad
fix fennec post-beetmover-dummy. r=bhearsum
Comment 15•7 years ago
|
||
bugherder |
Updated•6 years ago
|
Assignee: nobody → aki
You need to log in
before you can comment on or make changes to this bug.
Description
•