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.

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

OADP 3183 Doc improvements - Installing and configuring OADP & Uninst… #69508

Merged
merged 1 commit into from
Jan 9, 2024
Merged

OADP 3183 Doc improvements - Installing and configuring OADP & Uninst… #69508

merged 1 commit into from
Jan 9, 2024

Conversation

CarmiWisemon
Copy link
Contributor

@CarmiWisemon CarmiWisemon commented Dec 17, 2023 •
edited

OADP 1.3.0

OCP 4.13+

Resolves - https://issues.redhat.com/browse/OADP-3183

Deploy preview -

https://69508--ocpdocs-pr.netlify.app/openshift-enterprise/latest/backup_and_restore/application_backup_and_restore/installing/installing-oadp-aws#verifying-oadp-installation-1-2_installing-oadp-aws

Verification for 1.2 & earlier

https://69508--ocpdocs-pr.netlify.app/openshift-enterprise/latest/backup_and_restore/application_backup_and_restore/installing/installing-oadp-aws#verifying-oadp-installation-1-3_installing-oadp-aws

Verification for 1.3

https://69508--ocpdocs-pr.netlify.app/openshift-enterprise/latest/backup_and_restore/application_backup_and_restore/installing/about-installing-oadp#about-installing-oadp-on-multiple-namespaces_about-installing-oadp

Installation of OADP on multiple namespaces

https://69508--ocpdocs-pr.netlify.app/openshift-enterprise/latest/backup_and_restore/application_backup_and_restore/installing/about-installing-oadp#cpu-and-memory-requirement-for-configurations

CPU memory

@openshift-ci openshift-ci bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Dec 17, 2023
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Dec 17, 2023 •
edited

🤖 Tue Jan 09 09:49:58 - Prow CI generated the docs preview: https://69508--ocpdocs-pr.netlify.app

@CarmiWisemon CarmiWisemon force-pushed the oadp3183docimpvinstallingconfig branch 3 times, most recently from 7cb632d to b5a2c36 Compare December 21, 2023 11:36
@CarmiWisemon CarmiWisemon force-pushed the oadp3183docimpvinstallingconfig branch 2 times, most recently from 479983c to 52503e5 Compare December 31, 2023 09:00
@CarmiWisemon CarmiWisemon force-pushed the oadp3183docimpvinstallingconfig branch 3 times, most recently from 10c06be to e993d73 Compare January 7, 2024 09:45
@CarmiWisemon CarmiWisemon changed the title WIP - OADP 3183 Doc improvements - Installing and configuring OADP & Uninst… OADP 3183 Doc improvements - Installing and configuring OADP & Uninst… Jan 8, 2024
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 8, 2024
stillalearner
stillalearner approved these changes Jan 8, 2024
View reviewed changes
Copy link

@stillalearner stillalearner left a comment

Choose a reason for hiding this comment

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

lgtm

@CarmiWisemon CarmiWisemon force-pushed the oadp3183docimpvinstallingconfig branch from 83a529e to a3e8e0e Compare January 8, 2024 13:42
@CarmiWisemon
Copy link
Contributor Author

/label OADP
/label peer-review-needed

@openshift-ci openshift-ci bot added OADP Label for all OADP PRs peer-review-needed Signifies that the peer review team needs to review this PR labels Jan 8, 2024
@tmalove
Copy link
Contributor

tmalove commented Jan 8, 2024

/remove-label peer-review-needed
/label peer-review-in-progress

@openshift-ci openshift-ci bot added peer-review-in-progress Signifies that the peer review team is reviewing this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Jan 8, 2024
tmalove
tmalove approved these changes Jan 8, 2024
View reviewed changes
modules/oadp-velero-cpu-memory-requirements.adoc Outdated Show resolved Hide resolved
@tmalove
Copy link
Contributor

tmalove commented Jan 8, 2024

/remove-label peer-review-in-progress
/label peer-review-done

@openshift-ci openshift-ci bot 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 labels Jan 8, 2024
@CarmiWisemon CarmiWisemon force-pushed the oadp3183docimpvinstallingconfig branch from a3e8e0e to 4f34939 Compare January 9, 2024 09:27
@CarmiWisemon CarmiWisemon force-pushed the oadp3183docimpvinstallingconfig branch from 2dfd39a to 6dec6a2 Compare January 9, 2024 09:44
@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Jan 9, 2024

@CarmiWisemon: 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/test-infra repository. I understand the commands that are listed here.

@CarmiWisemon
Copy link
Contributor Author

/label merge-review-needed

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Jan 9, 2024
@lpettyjo lpettyjo added merge-review-in-progress Signifies that the merge review team is reviewing this PR ok-to-merge branch/enterprise-4.13 branch/enterprise-4.14 branch/enterprise-4.15 and removed merge-review-needed Signifies that the merge review team needs to review this PR merge-review-in-progress Signifies that the merge review team is reviewing this PR labels Jan 9, 2024
@lpettyjo lpettyjo added this to the Continuous Release milestone Jan 9, 2024
@lpettyjo lpettyjo merged commit 2ad2ec9 into openshift:main Jan 9, 2024
3 checks passed
@lpettyjo
Copy link
Contributor

lpettyjo commented Jan 9, 2024

/cherrypick enterprise-4.13

@lpettyjo
Copy link
Contributor

lpettyjo commented Jan 9, 2024

/cherrypick enterprise-4.14

@lpettyjo
Copy link
Contributor

lpettyjo commented Jan 9, 2024

/cherrypick enterprise-4.15

@openshift-cherrypick-robot
Copy link

@lpettyjo: #69508 failed to apply on top of branch "enterprise-4.13":

Patch is empty.
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To record the empty patch as an empty commit, run "git am --allow-empty".
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherrypick enterprise-4.13

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.

@openshift-cherrypick-robot
Copy link

@lpettyjo: new pull request created: #69976

In response to this:

/cherrypick enterprise-4.14

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.

This was referenced Jan 9, 2024
Closed
Merged
@openshift-cherrypick-robot
Copy link

@lpettyjo: new pull request created: #69977

In response to this:

/cherrypick enterprise-4.15

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.

This was referenced Jan 11, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stillalearner stillalearner stillalearner approved these changes

@tmalove tmalove tmalove approved these changes

Assignees
No one assigned
Labels
branch/enterprise-4.13 branch/enterprise-4.14 branch/enterprise-4.15 OADP Label for all OADP PRs ok-to-merge peer-review-done Signifies that the peer review team has reviewed this PR size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Milestone
Continuous Release
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@CarmiWisemon @ocpdocs-previewbot @tmalove @lpettyjo @openshift-cherrypick-robot @stillalearner