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

Use CMC as default config when set #50039

Merged
merged 4 commits into from
Dec 13, 2024
Merged

Conversation

hugoShaka
Copy link
Contributor

@hugoShaka hugoShaka commented Dec 10, 2024

Part of: RFD-184

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

This PR implements backward compatibility when CMC is specified. After this PR, if the user has no autoupdate_config resource but a cluster_maintenance_config resource from RFD 109, we will use the CMC to generate the config (update hour and update days) and craft the autoupdate_agent_rollout.

@hugoShaka hugoShaka added the no-changelog Indicates that a PR does not require a changelog entry label Dec 10, 2024
@hugoShaka hugoShaka force-pushed the hugo/default-config-from-cmc branch from faa086d to 20f0238 Compare December 10, 2024 21:05
lib/autoupdate/rollout/reconciler.go Outdated Show resolved Hide resolved
Part of: [RFD-184](#47126)

This commit implements backward compatibility when CMC is specified.
After this PR, if the user has no `autoupdate_config` resource but a
`cluster_maintenance_config` resource from RFD 109, we will use the CMC
to generate the config (update hour and update days) and craft the
`autoupdate_agent_rollout`.
@hugoShaka hugoShaka force-pushed the hugo/default-config-from-cmc branch from 20f0238 to 0b559ac Compare December 13, 2024 15:30
lib/autoupdate/rollout/reconciler.go Outdated Show resolved Hide resolved
lib/autoupdate/rollout/client_test.go Outdated Show resolved Hide resolved
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from strideynet December 13, 2024 18:05
@hugoShaka hugoShaka force-pushed the hugo/default-config-from-cmc branch from f062790 to 13f116d Compare December 13, 2024 20:22
@hugoShaka hugoShaka force-pushed the hugo/default-config-from-cmc branch from 13f116d to c877658 Compare December 13, 2024 20:25
@hugoShaka hugoShaka enabled auto-merge December 13, 2024 21:04
@hugoShaka hugoShaka added this pull request to the merge queue Dec 13, 2024
Merged via the queue into master with commit fe89b6d Dec 13, 2024
40 checks passed
@hugoShaka hugoShaka deleted the hugo/default-config-from-cmc branch December 13, 2024 21:28
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.

5 participants