#476 Added start time for supporting restarts for fixed rate schedules
Pull request opened by
pmahindrakar-oss
Signed-off-by: Prafulla Mahindrakar
prafulla.mahindrakar@gmail.com
TL;DR
• Updated the cron library to accept startTime for timedJob
• Using the lastExecution time when starting the fixed rate schedules
Testing Done:
• Registered the fixed rate schedule for 1 min rate
• Brought down the scheduler for a few hours
• Verified the catch up routine ran and caught up to current time using the last Execution timestamp
• Also verified the new routines are also respecting the last execution time
Type
☑︎ Bug Fix
☐ Feature
☐ Plugin
Are all requirements met?
☑︎ Code completed
☑︎ Smoke tested
☑︎ Unit tests added
☐ Code documentation added
☐ Any pending items have an associated Issue
Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
_Remove the '_fixes_' keyword if there will be multiple PRs to fix the linked issue_
fixes
flyteorg/flyte#2885
Follow-up issue
flyteorg/flyteadmin
GitHub Actions: Build & Push Flytescheduler Image
GitHub Actions: Goreleaser
GitHub Actions: Build & Push Flyteadmin Image
GitHub Actions: Bump Version
Codecov: 58.39% (-0.25%) compared to 00915ec
✅ 9 other checks have passed
9/14 successful checks