-
Notifications
You must be signed in to change notification settings - Fork 16.8k
[stable/atlantis] Make secret-webhook optional #13667
Conversation
Hi @wvidana. 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. |
/assign @lkysow |
@lkysow can you please review this PR? Or should I assign it to someone else? |
@wvidana sorry, this fell off my backlog, I'll take a look |
People might want to manage secrets on their own Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com>
Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com>
Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com>
Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com>
Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com>
Thanks for the review @lkysow ! Changes are ready again for revision. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lkysow, wvidana 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 |
* Make the template secret-webhook optional People might want to manage secrets on their own Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com> * Bump chart version Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com> * Using a variable to define the reference to the secret Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com> * Renaming secretWebhookName to vcsSecretsName Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com> * New feature version Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com>
* Make the template secret-webhook optional People might want to manage secrets on their own Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com> * Bump chart version Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com> * Using a variable to define the reference to the secret Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com> * Renaming secretWebhookName to vcsSecretsName Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com> * New feature version Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com>
Volunteering from issue helm#13867 Past contributions to cerebro chart: - helm#14506 Past contributions to other helm charts: - helm#14196 - helm#14189 - helm#13667 - helm#13666 - helm#13665
Volunteering from issue helm#13867 Past contributions to cerebro chart: - helm#14506 Past contributions to other helm charts: - helm#14196 - helm#14189 - helm#13667 - helm#13666 - helm#13665 Signed-off-by: Wilfrido Vidana <wilfrido.vidana@omf.com>
* Adding wvidana as collaborator Volunteering from issue #13867 Past contributions to cerebro chart: - #14506 Past contributions to other helm charts: - #14196 - #14189 - #13667 - #13666 - #13665 Signed-off-by: Wilfrido Vidana <wilfrido.vidana@omf.com> * Adding the OWNERS file to .helmignore Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com>
* Adding wvidana as collaborator Volunteering from issue helm#13867 Past contributions to cerebro chart: - helm#14506 Past contributions to other helm charts: - helm#14196 - helm#14189 - helm#13667 - helm#13666 - helm#13665 Signed-off-by: Wilfrido Vidana <wilfrido.vidana@omf.com> * Adding the OWNERS file to .helmignore Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com>
* Adding wvidana as collaborator Volunteering from issue helm#13867 Past contributions to cerebro chart: - helm#14506 Past contributions to other helm charts: - helm#14196 - helm#14189 - helm#13667 - helm#13666 - helm#13665 Signed-off-by: Wilfrido Vidana <wilfrido.vidana@omf.com> * Adding the OWNERS file to .helmignore Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com>
* Adding wvidana as collaborator Volunteering from issue helm#13867 Past contributions to cerebro chart: - helm#14506 Past contributions to other helm charts: - helm#14196 - helm#14189 - helm#13667 - helm#13666 - helm#13665 Signed-off-by: Wilfrido Vidana <wilfrido.vidana@omf.com> * Adding the OWNERS file to .helmignore Signed-off-by: Wilfrido Vidana <wvidanas@gmail.com>
What this PR does / why we need it:
People might want to manage secrets on their own. This PR makes the secret-webhook from the templates optional if no git credentials are given.
Which issue this PR fixes
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
[stable/chart]