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

🌱 Cleanup v1 cluster before creating v3 clusters on upgrade #576

Conversation

Danil-Grigorev
Copy link
Contributor

@Danil-Grigorev Danil-Grigorev commented Jun 20, 2024

What this PR does / why we need it:

Perform automatic provisioning cluster removal on upgrade with v3 clusters feature flag enabled.
In case of a duplicate cluster entries, a user can re-run the upgrade without settings change.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:

Checklist:

  • squashed commits into logical changes
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

@Danil-Grigorev Danil-Grigorev requested a review from a team as a code owner June 20, 2024 08:33
@Danil-Grigorev Danil-Grigorev force-pushed the cleanup-v1-cluster-after-v3-migration branch 4 times, most recently from 8d7169f to 9b1f21a Compare June 21, 2024 08:36
salasberryfin
salasberryfin previously approved these changes Jun 21, 2024
Copy link
Contributor

@salasberryfin salasberryfin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Danil-Grigorev. LGTM

Copy link
Contributor

@furkatgofurov7 furkatgofurov7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, few nits below:

@furkatgofurov7 furkatgofurov7 mentioned this pull request Jun 24, 2024
4 tasks
Signed-off-by: Danil-Grigorev <danil.grigorev@suse.com>
- Add e2e tests, verifying the added migration functionality

Signed-off-by: Danil-Grigorev <danil.grigorev@suse.com>
Signed-off-by: Danil-Grigorev <danil.grigorev@suse.com>
@Danil-Grigorev Danil-Grigorev force-pushed the cleanup-v1-cluster-after-v3-migration branch from 4fcec14 to 78d3b6c Compare June 24, 2024 11:46
@Danil-Grigorev Danil-Grigorev merged commit 80e434b into rancher:main Jun 24, 2024
9 checks passed
@Danil-Grigorev Danil-Grigorev mentioned this pull request Jul 16, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants