-
Notifications
You must be signed in to change notification settings - Fork 79
Backup and setup automation #2601
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
Conversation
Signed-off-by: Ben Luzarraga <[email protected]>
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bluzarraga The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
…herwise create if operator already installed Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
…ployment function Signed-off-by: Ben Luzarraga <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
Successful E2E test on two fresh clusters so this PR is ready for review. Methodology:
Especially if zen is installed, it can take time for the entire script to complete so be sure to give it time. |
Tested in the cluster, backup succeed
|
Hi @bluzarraga, I hit an issue when testing restore
I think we need to declare two more variables |
Signed-off-by: Allen Li <[email protected]>
Signed-off-by: Allen Li <[email protected]>
/lgtm tested and verified in the cluster, thank you @bluzarraga for this pr |
What this PR does / why we need it: Automates the installation and setup of OADP on both the backup and restore cluster (if specified), preps the backup cluster for backup by deploying necessary BR resources and labeling CPFS resources, then executes and validates the backup.
Which issue(s) this PR fixes:
Fixes # https://github.ibm.com/IBMPrivateCloud/roadmap/issues/67462 & https://github.ibm.com/IBMPrivateCloud/roadmap/issues/67461
Special notes for your reviewer:
How to backport this PR to other branch:
backport <branch-name>
backport <branch-name>
and leave a comment/backport
to trigger the backport action