Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

set rollout start date and don't start updating if rollout just changed #50365

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

hugoShaka
Copy link
Contributor

@hugoShaka hugoShaka commented Dec 17, 2024

Part of: RFD-184

Goal (internal): https://github.com/gravitational/cloud/issues/10289

This PR does two changes:

  • the controller now sets the rollout start time when resetting the rollout
  • the controller will not start a group if the rollout changed during the maintenance window (checks if the rollout start time is in the window)

@hugoShaka hugoShaka force-pushed the hugo/rollout-start-date branch 2 times, most recently from 33d8eb0 to 47ce2f3 Compare December 18, 2024 18:19
@hugoShaka hugoShaka marked this pull request as ready for review December 18, 2024 18:19
@hugoShaka hugoShaka force-pushed the hugo/rollout-start-date branch from 47ce2f3 to 6d855e5 Compare December 18, 2024 18:19
@hugoShaka hugoShaka added the no-changelog Indicates that a PR does not require a changelog entry label Dec 18, 2024
@github-actions github-actions bot requested review from bernardjkim and zmb3 December 18, 2024 18:20
This commit does two changes:
- the controller now sets the rollout start time when resetting the
  rollout
- the controller will not start a group if the rollout changed during
  the maintenance window (checks if the rollout start time is in the
  window)
@hugoShaka hugoShaka force-pushed the hugo/rollout-start-date branch from ee1ce53 to e285c9e Compare December 19, 2024 19:37
@hugoShaka hugoShaka enabled auto-merge December 19, 2024 23:19
@hugoShaka hugoShaka added this pull request to the merge queue Dec 19, 2024
Merged via the queue into master with commit aaea047 Dec 19, 2024
44 checks passed
@hugoShaka hugoShaka deleted the hugo/rollout-start-date branch December 19, 2024 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Indicates that a PR does not require a changelog entry size/md
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants