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

ci/cli: add full backup/restore test #1561

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

ldevulder
Copy link
Contributor

@ldevulder ldevulder commented Sep 13, 2024

This test simulates migration of a Rancher Manager server. This PR will fix #1212.

Note: this version doesn't install worker nodes because there is an issue as we need to do a separate etcd backup/restore to be able to deploy new nodes. This will be done in another PR (etcd backup/restore and provisioning of worker nodes). The operator uninstallation test is also not working because the operator is not installed by helm after a restore, something to investigate further if it's expected or not.

Verification runs:

@ldevulder ldevulder self-assigned this Sep 13, 2024
@ldevulder ldevulder requested a review from a team as a code owner September 13, 2024 13:33
@ldevulder ldevulder marked this pull request as draft September 13, 2024 13:33
@ldevulder ldevulder force-pushed the ci-cli-add-full-backup-restore-test branch 23 times, most recently from 56f6870 to 04aa0ac Compare September 20, 2024 13:38
@ldevulder ldevulder force-pushed the ci-cli-add-full-backup-restore-test branch 2 times, most recently from 331c458 to 3405cc2 Compare September 23, 2024 09:04
@ldevulder ldevulder marked this pull request as ready for review September 23, 2024 09:09
@ldevulder ldevulder force-pushed the ci-cli-add-full-backup-restore-test branch from 3405cc2 to a44ef79 Compare September 23, 2024 09:23
This test simulates migration of a Rancher Manager server.

Signed-off-by: Loic Devulder <[email protected]>
@ldevulder ldevulder force-pushed the ci-cli-add-full-backup-restore-test branch from a44ef79 to 33e728d Compare September 23, 2024 09:32
Copy link
Contributor

@juadk juadk left a comment

Choose a reason for hiding this comment

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

LGTM! nice one thx

@ldevulder ldevulder merged commit da37235 into main Sep 23, 2024
3 checks passed
@ldevulder ldevulder deleted the ci-cli-add-full-backup-restore-test branch September 23, 2024 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

ci: add automation for a full backup/restore test (aka. cluster migration)
2 participants