feat: add publish of helm chart to ghcr.io#2853
feat: add publish of helm chart to ghcr.io#2853a1994sc wants to merge 6 commits intozalando:masterfrom
Conversation
Signed-off-by: Allen Conlon <software@conlon.dev>
a1994sc
requested review from
FxKu,
Jan-M,
hughcapet,
idanovinda,
jopadi,
macedigital and
sdudoladov
as code owners
|
@FxKu can I please get an update on this PR? |
|
Cannot start a pipeline due to: Click on pipeline status check Details link below for more information. |
|
@FxKu sorry for the ping, but can I get an update for this PR? |
| run: | | ||
| helm package charts/postgres-operator-ui | ||
| helm push postgres-operator-ui-*.tgz oci://${{ env.REGISTRY }}/zalando/charts | ||
| rm -rf postgres-operator-ui-*.tgz |
There was a problem hiding this comment.
Will this work out of the box? I would assume helm needs to be installed first fr this GH workflow step to work.
There was a problem hiding this comment.
Helm is installed by default on the github runners:
https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2404-Readme.md
There was a problem hiding this comment.
@a1994sc Sorry, helm is usually not a priority for us. Don't you need a login step like helm registry login ... or is this already covered with the existing login step?
There was a problem hiding this comment.
Helm will use the docker config setup by the docker/login-action action you use to publish the docker images
|
@FxKu should I close this PR? or can I get an update on what is preventing it from getting a review? |
3 participants
This PR adds actions to publish the
postgres-operatorandpostgres-opeator-uihelm charts to the github container registry