Closed
Bug 1381669
Opened 7 years ago
Closed 7 years ago
port backfill to actions.json
Categories
(Taskcluster :: Services, enhancement)
Taskcluster
Services
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla56
People
(Reporter: bstack, Assigned: bstack)
References
Details
Attachments
(1 file, 1 obsolete file)
No description provided.
Assignee | ||
Comment 1•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=85a96e5518b0
Assignee | ||
Comment 2•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=9be46718ce5a
Comment hidden (mozreview-request) |
Assignee | ||
Comment 4•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=a8fa4c13c10b
Assignee | ||
Comment 5•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=f8c9db943e00
Comment hidden (mozreview-request) |
Comment 7•7 years ago
|
||
mozreview-review |
Comment on attachment 8888063 [details] * Bug 1381669 - Port backfilling to actions.json https://reviewboard.mozilla.org/r/158948/#review165316 ::: taskcluster/actions/backfill.py:26 (Diff revision 2) > + > +@register_callback_action( > + title='Backfill', > + name='backfill', > + symbol='Bk', > + description='Adds a specific job to previous pushes.', "Runs this job on previous pushes" -- it took me a minute to figure out *which* specific job.. ::: taskcluster/actions/backfill.py:57 (Diff revision 2) > + pushlog_url = PUSHLOG_TMPL.format(parameters['head_repository'], start_id, end_id) > + r = requests.get(pushlog_url) > + r.raise_for_status() > + pushes = pushes + r.json()['pushes'].keys() > + if len(pushes) >= depth: > + break I think you want to decrement `end_id` here? ::: taskcluster/actions/backfill.py:61 (Diff revision 2) > + if len(pushes) >= depth: > + break > + pushes = sorted(pushes)[-depth:] > + > + r = requests.get(pushlog_url) > + r.raise_for_status() Why request this again? ::: taskcluster/actions/util.py:36 (Diff revision 2) > label = task_def['metadata']['name'] > session = get_session() > create.create_task(session, task_id, label, task_def) > + > + > +def clone_and_update_task(task, deadline, expires): We talked about this in irc. I think you can just use some of the `taskgraph.util.parameterization` stuff I made in my patch.
Attachment #8888063 -
Flags: review?(dustin)
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Attachment #8888063 -
Attachment is obsolete: true
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Attachment #8891103 -
Flags: review?(dustin)
Comment 10•7 years ago
|
||
mozreview-review |
Comment on attachment 8891103 [details] Bug 1381669 - Add backfill as an actions.json task https://reviewboard.mozilla.org/r/162282/#review167828
Attachment #8891103 -
Flags: review?(dustin) → review+
Comment 11•7 years ago
|
||
Pushed by dmitchell@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c2c67f39a75e Add backfill as an actions.json task r=dustin
Comment 12•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/159e20b9dd04c941f3e48f6c90586bebb94c7a08 Bug 1381669 - Add backfill as an actions.json task r=dustin
Comment 13•7 years ago
|
||
Backout by kwierso@gmail.com: https://hg.mozilla.org/integration/autoland/rev/20ca66b2d539 Backed out changeset c2c67f39a75e
Comment 14•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/159e20b9dd04
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Updated•5 years ago
|
Component: Integration → Services
You need to log in
before you can comment on or make changes to this bug.
Description
•