Page MenuHomePhabricator

Restart running schedules when updated
ClosedPublic

Authored by sashank on Thu, Oct 3, 3:27 AM.

Details

Reviewers
alangenfeld
Group Reviewers
Restricted Project
Commits
R1:bc8bd2e9facd: Restart running schedules when updated
Summary

Previously, when a ScheduleDefinition's cron_schedule was updated, the metadata files were properly updated but the crob jobs for running schedules were not.

Now, when up is called, updated running schedules are restarted to ensure the cron tab is up to date.

Test Plan
$ dagster schedule up
$ dagster schedule start many_events_every_min
// modify schedule
$ dagster schedule up
// verify cron job schedule is updated

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

sashank created this revision.Thu, Oct 3, 3:27 AM
alangenfeld accepted this revision.Thu, Oct 3, 3:08 PM
alangenfeld added a subscriber: alangenfeld.

whynot

This revision is now accepted and ready to land.Thu, Oct 3, 3:08 PM
sashank updated this revision to Diff 5497.EditedThu, Oct 3, 5:44 PM
This comment has been deleted.