Skip to content

[OSDOCS-11124]: Add automated backup/restore with OADP docs #94958

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 11, 2025
Merged

[OSDOCS-11124]: Add automated backup/restore with OADP docs #94958

merged 1 commit into from
Jul 11, 2025

Conversation

lahinson
Copy link
Contributor

@lahinson lahinson commented Jun 18, 2025 •
edited
Loading

@lahinson lahinson added this to the Continuous Release milestone Jun 18, 2025
@openshift-ci openshift-ci bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jun 18, 2025
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Jun 18, 2025 •
edited
Loading

🤖 Fri Jul 11 15:51:03 - Prow CI generated the docs preview:

https://94958--ocpdocs-pr.netlify.app/openshift-enterprise/latest/hosted_control_planes/hcp_high_availability/hcp-disaster-recovery-oadp-auto.html
https://94958--ocpdocs-pr.netlify.app/openshift-enterprise/latest/hosted_control_planes/hcp_high_availability/hcp-disaster-recovery-oadp.html

@lahinson lahinson force-pushed the osdocs-11124-hcp-automate-capture branch 7 times, most recently from aad960c to b430ae0 Compare June 18, 2025 18:31
lahinson
lahinson commented Jun 18, 2025
View reviewed changes
jparrill
jparrill approved these changes Jun 19, 2025
View reviewed changes
Copy link

@jparrill jparrill left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 19, 2025
@lahinson lahinson force-pushed the osdocs-11124-hcp-automate-capture branch from b430ae0 to 30fccd3 Compare June 20, 2025 14:39
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Jun 20, 2025
@lahinson lahinson force-pushed the osdocs-11124-hcp-automate-capture branch from 30fccd3 to c50093e Compare June 20, 2025 14:52
@lahinson
Copy link
Contributor Author

@LiangquanLi930 When you're available, please provide QE review. Thanks!

@lahinson lahinson added the peer-review-needed Signifies that the peer review team needs to review this PR label Jun 20, 2025
@mburke5678 mburke5678 added the peer-review-in-progress Signifies that the peer review team is reviewing this PR label Jun 20, 2025
mburke5678
mburke5678 reviewed Jun 20, 2025
View reviewed changes
modules/hcp-dr-oadp-backup-cp-workload-auto.adoc Outdated

.Next steps

* Restoring a hosted cluster by using OADP
Copy link
Contributor

Choose a reason for hiding this comment

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

Need a link here?

Suggested change
* Restoring a hosted cluster by using OADP
* Restoring a hosted cluster by using {oadp-short}

@mburke5678
Copy link
Contributor

mburke5678 commented Jun 20, 2025 •
edited
Loading

@lahinson I added some comments. Let me know if you need explanation or further info. Otherwise LGTM. Sorry it took so long!

@mburke5678 mburke5678 added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-in-progress Signifies that the peer review team is reviewing this PR peer-review-needed Signifies that the peer review team needs to review this PR labels Jun 20, 2025
@lahinson lahinson force-pushed the osdocs-11124-hcp-automate-capture branch 8 times, most recently from 29c2a23 to e19e6e5 Compare June 23, 2025 18:56
@LiangquanLi930
Copy link
Member

@geliu2016 Could you help to review again?

@geliu2016
Copy link

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 25, 2025
@lahinson lahinson force-pushed the osdocs-11124-hcp-automate-capture branch from e19e6e5 to 430d92e Compare July 11, 2025 15:40
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Jul 11, 2025
@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Jul 11, 2025

New changes are detected. LGTM label has been removed.

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Jul 11, 2025

@lahinson: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@lahinson lahinson merged commit 1426ebe into openshift:main Jul 11, 2025
2 checks passed
@lahinson
Copy link
Contributor Author

/cherrypick enterprise-4.19

@lahinson
Copy link
Contributor Author

/cherrypick enterprise-4.20

@openshift-cherrypick-robot
Copy link

@lahinson: new pull request created: #96015

In response to this:

/cherrypick enterprise-4.19

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Jul 11, 2025
Open
@openshift-cherrypick-robot
Copy link

@lahinson: new pull request created: #96016

In response to this:

/cherrypick enterprise-4.20

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Jul 11, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mburke5678 mburke5678 mburke5678 left review comments

@jparrill jparrill jparrill approved these changes

Assignees

@jparrill jparrill

@geliu2016 geliu2016

Labels
branch/enterprise-4.19 branch/enterprise-4.20 ok-to-merge peer-review-done Signifies that the peer review team has reviewed this PR size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
Continuous Release
Development

Successfully merging this pull request may close these issues.
7 participants
@lahinson @ocpdocs-previewbot @mburke5678 @LiangquanLi930 @geliu2016 @openshift-cherrypick-robot @jparrill