Closed
Bug 1453067
Opened 6 years ago
Closed 6 years ago
Taskcluster cron should support days
Categories
(Firefox Build System :: Task Configuration, task)
Firefox Build System
Task Configuration
Tracking
(firefox-esr60 fixed, firefox61 fixed)
RESOLVED
FIXED
mozilla61
People
(Reporter: sfraser, Assigned: sfraser)
References
Details
Attachments
(3 files)
We have a need to run scheduled tasks less frequently than once a day, and taskcluster cron currently only supports hours and minutes as options.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → sfraser
Comment 1•6 years ago
|
||
We need to run things less often than once a day, so adding support for days to taskcluster cron. 'day' is the day of the month, 'weekday' is used as a datetime.weekday (not isoweekday), or a string comparable to strftime('%A') or strftime('%a')
Updated•6 years ago
|
Component: General → Task Configuration
Product: Taskcluster → Firefox Build System
Assignee | ||
Updated•6 years ago
|
Attachment #8966675 -
Flags: review?(dustin)
Updated•6 years ago
|
Attachment #8966675 -
Flags: review?(dustin) → review-
Assignee | ||
Comment 2•6 years ago
|
||
What were the improvements needed for the update?
Flags: needinfo?(dustin)
Comment 4•6 years ago
|
||
Comment 5•6 years ago
|
||
Oh, sorry, I thought Phabricator wasn't showing those to you. I didn't see that you had updated the patch, or that you had made comments. For whatever reason, it doesn't show me your comments inline, although it shows me my comments inline. Egads, but this is unusable. I think this looks good now.
Updated•6 years ago
|
Attachment #8966675 -
Flags: review- → review+
Assignee | ||
Comment 6•6 years ago
|
||
Comment on attachment 8966675 [details] Bug 1453067 Support for days (of week or month) in taskcluster cron r=dustin One more update - I'd neglected to remove the tests for the numeric weekday last night, and adjust the elif condition in util.py
Attachment #8966675 -
Flags: review+ → review?(dustin)
Comment 7•6 years ago
|
||
Comment on attachment 8966675 [details] Bug 1453067 Support for days (of week or month) in taskcluster cron r=dustin Dustin J. Mitchell [:dustin] pronoun: he has approved the revision. https://phabricator.services.mozilla.com/D903
Attachment #8966675 -
Flags: review+
Updated•6 years ago
|
Attachment #8966675 -
Flags: review?(dustin)
Attachment #8966675 -
Flags: review+
Pushed by sfraser@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/5ea0755f98af Support for days (of week or month) in taskcluster cron r=dustin
Comment 9•6 years ago
|
||
Comment on attachment 8966675 [details] Bug 1453067 Support for days (of week or month) in taskcluster cron r=dustin Dustin J. Mitchell [:dustin] pronoun: he has approved the revision. https://phabricator.services.mozilla.com/D903
Attachment #8966675 -
Flags: review+
Comment 10•6 years ago
|
||
lol phabricator's a little late to the party here
Comment 11•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/5ea0755f98af
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Comment 12•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-esr60/rev/6809bbed542c
status-firefox-esr60:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•