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
OADP-205: Remove registry deployment in favor of plugin enhancement #743
Conversation
Skipping CI for Draft Pull Request. |
Codecov Report
@@ Coverage Diff @@
## master #743 +/- ##
==========================================
- Coverage 36.96% 34.39% -2.58%
==========================================
Files 16 16
Lines 3219 2826 -393
==========================================
- Hits 1190 972 -218
+ Misses 1926 1763 -163
+ Partials 103 91 -12
Continue to review full report at Codecov.
|
/test all |
oops forgot needs plugin PR merge first |
ready for review but |
/hold |
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.
Operator changes look straight forward to me, added some NITs, Thanks @kaovilai
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.
This PR also needs to include changes to the CSV to remove the registry as a related image.
Removed registry from CSV as a relatedImage. |
updating quay bundle ... |
bundle is up to date |
/retest-required |
/retest |
/test 4.9-operator-e2e-gcp |
For AWS:
For GCP
https://github.com/distribution/distribution/blob/3f4c558dac40dfb4fe13285ffc43a0d738d3a3f0/registry/storage/driver/gcs/gcs.go#L80
|
0fd1b54
to
db46910
Compare
openshift/openshift-velero-plugin#152 might be an issue. |
waiting on https://app.travis-ci.com/github/openshift/openshift-velero-plugin/builds/253057305 to retest again |
Looks like no retest required.. since all running tests are still installing ocp |
/test 4.10-operator-e2e-aws |
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.
Code changes LGTM, will ack when E2E is updated to disable iamge backup for GCP
03d1e8e
to
a8e4b52
Compare
GCP tests blocked by openshift/openshift-velero-plugin#155. Once travis builds plugin image, retesting GCP is expected to pass. |
/retest |
@kaovilai: 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. |
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
/unhold |
This velero plugin registry enhancements will do the following
Install this PR operator with
Instruction for testing this PR prior to updated latest image at openshift/openshift-velero-plugin#145
OADP-205