Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

fix/msp: test for cron interval changes based on time, add more restrictions#62969

Merged
bobheadxi merged 1 commit into
mainfrom
msp-spec-cron-schedule-estimation
May 30, 2024
Merged

fix/msp: test for cron interval changes based on time, add more restrictions#62969
bobheadxi merged 1 commit into
mainfrom
msp-spec-cron-schedule-estimation

Conversation

@bobheadxi

Copy link
Copy Markdown
Member

Addresses problem noticed in https://github.com/sourcegraph/managed-services/pull/1486#issuecomment-2137887423

Test plan

Unit tests

Changelog

  • Fixed an issue with output of sg msp generate for MSP jobs with particular schedules changing throughout the week
  • MSP jobs schedules now must be between 15 minutes at the most frequent, and every week at the least frequent

@bobheadxi bobheadxi requested review from a team and jac May 29, 2024 18:24
@cla-bot cla-bot Bot added the cla-signed label May 29, 2024
@bobheadxi bobheadxi merged commit cb62afa into main May 30, 2024
@bobheadxi bobheadxi deleted the msp-spec-cron-schedule-estimation branch May 30, 2024 01:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants