Add Helm chart OCI release to GH automation #340
Conversation
cert-manager-prow
bot
added
dco-signoff: yes
inteon
force-pushed
the
add_helm_oci_release
branch
3 times, most recently
from
6651e31 to
65f6b82
Compare
There was a problem hiding this comment.
Thanks for also updating the RELEASE docs.
How does charts.jetstack.io get updated, if not by a PR?
/approve
/lgtm
There was a problem hiding this comment.
It's not yet fully automated, but the initial automation is this: https://github.com/jetstack/jetstack-charts/pull/378
The goal is to run this automation periodically.
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: wallrj 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 |
1 similar comment
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: wallrj 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 |
cert-manager-prow
bot
added
the
approved
Signed-off-by: Tim Ramlot <[email protected]>
inteon
force-pushed
the
add_helm_oci_release
branch
from
2b84a96 to
2147324
Compare
cert-manager-prow
bot
removed
the
lgtm
|
New changes are detected. LGTM label has been removed. |
|
/hold |
cert-manager-prow
bot
added
the
do-not-merge/hold
Signed-off-by: Tim Ramlot <[email protected]>
inteon
force-pushed
the
add_helm_oci_release
branch
from
2147324 to
d9b7298
Compare
cert-manager-prow
bot
removed
the
lgtm
|
New changes are detected. LGTM label has been removed. |
inteon
added
lgtm
Adds logic for pushing the Helm chart to an OCI registry (and adding a non-v-prefixed tag)
See cert-manager/makefile-modules#219 for more info.