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
OCPBUGS-34399: Exposing chunksize variable to utilize docker registry config #1026
base: master
Are you sure you want to change the base?
Conversation
Skipping CI for Draft Pull Request. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: deepsm007 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/test all |
b9281ad
to
b84782f
Compare
/test all |
so far so good, I'll put it on hold until the o/api changes merge (your hold didn't take). |
8471e70
to
aaa90f5
Compare
547c9f5
to
d6af07a
Compare
e913bad
to
b34aad0
Compare
7ef75ca
to
89c7bdc
Compare
/retest-required |
/retest |
89c7bdc
to
fe0da06
Compare
unit test failing:
since this env var is optional, should the test really be expecting it? |
Looks like there might be some permissions missing from the operator as well, seeing this in the logs:
|
fe0da06
to
933c26e
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.
Where is the change that actually adds the chunksize env var to image registry pod spec? I think maybe that's why that unit test was failing? In that case, maybe we should add it back (sorry...)?
933c26e
to
93419c7
Compare
/test all |
93419c7
to
c3118ab
Compare
c3118ab
to
0be3c8c
Compare
/test e2e-aws-ovn-image-registry |
0be3c8c
to
a62f325
Compare
/unhold |
@deepsm007: This pull request references Jira Issue OCPBUGS-34399, which is valid. The bug has been moved to the POST state. 3 validation(s) were run on this bug
Requesting review from QA contact: The bug has been updated to refer to the pull request using the external bug tracker. 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 openshift-eng/jira-lifecycle-plugin repository. |
@deepsm007: This pull request references Jira Issue OCPBUGS-34399, which is valid. 3 validation(s) were run on this bug
Requesting review from QA contact: 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 openshift-eng/jira-lifecycle-plugin repository. |
/retest |
1 similar comment
/retest |
@deepsm007: The following tests failed, say
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. |
Trying to work around a bug for r2 integration with s3 bucket, exposing chunksize variable by utilizing the docker registry S3 driver configuration option: https://docker-docs.uclv.cu/registry/storage-drivers/s3/
/hold for openshift/api#1864 to merge
https://issues.redhat.com//browse/IR-471
/cc @flavianmissi