-
Notifications
You must be signed in to change notification settings - Fork 17
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
[Securesign 297] Backup and Restore Docs #277
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just some small things that stood out to, looks good overall. :)
Should: docs/Scheduled-Backup.md be empty?
# Backup Procedure | ||
## Prerequisites | ||
- Successfully installed and configured OADP Operator. | ||
- See OADP-Install.md. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should link to the OADP-Install.md
Hi @tommyd450, I unable to restore TAS instance because OADP will not recreate deleted PVCs. I did these steps:
|
a450020
to
0b1c99c
Compare
11f9f2c
to
a1a7664
Compare
a1a7664
to
fca847e
Compare
fca847e
to
5af5eb2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: osmman, tommyd450 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 |
/override ci/prow/tas-operator-e2e |
@osmman: Overrode contexts on behalf of osmman: ci/prow/tas-operator-e2e In response to this:
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/test-infra repository. |
First draft of these if someone has the cycles to give this a test please ping me here or on slack with questions.