-
Notifications
You must be signed in to change notification settings - Fork 16.8k
[incubator/registry-creds] Add registry-creds chart #13660
Conversation
Signed-off-by: Andres Alvarez <[email protected]>
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: kir4h If they are not already assigned, you can assign the PR to them by writing 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 |
Hi @kir4h. 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: Andres Alvarez <[email protected]>
/assign @prydonius |
@prydonius anything else I can do to ease up the reviewal? |
hey @kir4h Have you considered hosting this chart and adding it to helm hub? |
Nope, I'm not the owner of I noticed lot of projects moving out and being hosted independently; is this being encouraged from |
a bit of both. check this out: |
Closing PR as the chart is now hosted in Github pages and discoverable through Helm hub (helm/hub#88) |
Signed-off-by: Andres Alvarez [email protected]
What this PR does / why we need it:
This PR creates a chart to bootstraps a registry-creds deployment on a Kubernetes cluster
Special notes for your reviewer:
This is the first chart I create so any feedback is welcome. I have tried to follow the suggested guides as much as possible from my knowledge.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
[stable/chart]