[stable/openebs] Add the opportunity to change docker registry #22372
[stable/openebs] Add the opportunity to change docker registry #22372
Conversation
Signed-off-by: olivier beyler <olivier.beyler@orange.com>
Signed-off-by: olivier beyler <olivier.beyler@orange.com>
Hi @obeyler. Thanks for your PR. I'm waiting for a helm member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
Signed-off-by: olivier beyler <olivier.beyler@orange.com>
Signed-off-by: olivier beyler <olivier.beyler@orange.com>
/ok-to-test |
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
/approved
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: kmova, obeyler 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 |
…22372) * Add the opportunity to change docker registry Signed-off-by: olivier beyler <olivier.beyler@orange.com> * Add parameter in README.md Signed-off-by: olivier beyler <olivier.beyler@orange.com> * fixup missing changement Signed-off-by: olivier beyler <olivier.beyler@orange.com> * fixup defaut value in readme Signed-off-by: olivier beyler <olivier.beyler@orange.com>
…22372) * Add the opportunity to change docker registry Signed-off-by: olivier beyler <olivier.beyler@orange.com> * Add parameter in README.md Signed-off-by: olivier beyler <olivier.beyler@orange.com> * fixup missing changement Signed-off-by: olivier beyler <olivier.beyler@orange.com> * fixup defaut value in readme Signed-off-by: olivier beyler <olivier.beyler@orange.com>
…22372) * Add the opportunity to change docker registry Signed-off-by: olivier beyler <olivier.beyler@orange.com> * Add parameter in README.md Signed-off-by: olivier beyler <olivier.beyler@orange.com> * fixup missing changement Signed-off-by: olivier beyler <olivier.beyler@orange.com> * fixup defaut value in readme Signed-off-by: olivier beyler <olivier.beyler@orange.com> Signed-off-by: Adrien Loiseau <adrien.loiseau@logic-immo.com>
…22372) * Add the opportunity to change docker registry Signed-off-by: olivier beyler <olivier.beyler@orange.com> * Add parameter in README.md Signed-off-by: olivier beyler <olivier.beyler@orange.com> * fixup missing changement Signed-off-by: olivier beyler <olivier.beyler@orange.com> * fixup defaut value in readme Signed-off-by: olivier beyler <olivier.beyler@orange.com> Signed-off-by: Miguel Mingorance <miguel.mingorance@deliveryhero.com>
Signed-off-by: olivier beyler olivier.beyler@orange.com
Is this a new chart
no
What this PR does / why we need it:
In case of airgap, it's frequent to use a private docker registry and not neither docker.io or quay.io
It give the opportunity to change the docker repository from quai.io to private repository in one shot
and evict to have to change all docker image in the helm chart
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
[stable/mychartname]
)