Closed Bug 1425985 Opened 2 years ago Closed 2 years ago

Error out when a task exceeds MAX_DEPENDENCIES.

Categories

(Release Engineering :: Release Automation: Other, enhancement)

enhancement
Not set

Tracking

(firefox59 fixed)

RESOLVED FIXED
Tracking Status
firefox59 --- fixed

People

(Reporter: aki, Assigned: aki)

References

Details

Attachments

(4 files)

No description provided.
Attachment #8937567 - Flags: review?(aki) → review+
Comment on attachment 8937567 [details]
bug 1425985 - Add check for max dependencies.

https://reviewboard.mozilla.org/r/208246/#review214008
We should be able to autoland these patches once they're all r+'ed.
Dustin, Callek - just a heads up about these changes. They're largely relpro, other than MAX_DEPENDENCIES and the associated test in task.py.
(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 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 on attachment 8937569 [details]
bug 1425985 - fix beetmover-cdns deps.

https://reviewboard.mozilla.org/r/208250/#review214152
Attachment #8937569 - Flags: review?(bhearsum) → 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+
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
Assignee: nobody → aki
You need to log in before you can comment on or make changes to this bug.