appany / helm-oci-chart-releaser

Push Helm Charts to OCI-based registries
MIT License
59 stars 13 forks source link

Broken with Helm 3.7.0 Release #4

Closed sbckr closed 2 years ago

sbckr commented 2 years ago

Hi, first of all, thank you for providing this Action.

Describe the bug With Helm Release 3.7.0 the helm chart command was removed or partially replaced by other commands:

Notable Changes

A few major changes were made to the experimental OCI feature:

  • helm chart export has been removed
  • helm chart list has been removed
  • helm chart pull has been replaced with helm pull
  • helm chart push has been replaced with helm push
  • helm chart remove has been removed
  • helm chart save has been replaced with helm package

Since GitHub is already shipping Helm 3.7.0 with their latest Ubuntu 20.04 virtual-environment, running the helm-oci-chart-releaser fails with the following error:

Error: unknown command "chart" for "helm"
Run 'helm --help' for usage.
Error: Process completed with exit code 1.

To Reproduce Steps to reproduce the behavior:

  1. Define GitHub workflow using GitHub virtual-environment ubuntu-20.04
  2. Define appany/helm-oci-chart-releaser@v0.2.0 as a wotkflow step

Or checkout one of our repository's workflow (just default stuff):

Expected behavior The action should complete (upload the chart) as intended.

Screenshots image

sergeyshaykhullin commented 2 years ago

@sbckr Thanks for your detailed report!

Seems like helm stabilized oci apis and if @v0.3.0 version will work as expected i will release @v1

sbckr commented 2 years ago

@sergeyshaykhullin Thank you for your quick response! 🏎️

I'll adapt our pipeline to use release v0.3.0 and provide further feedback once available.

sergeyshaykhullin commented 2 years ago

@sbckr Action is migrated to helm 3.7.0, tested with ghcr. But some cd tools may not already adopted new helm oci format.

I will not release @v1 as helm oci support is still experimental

sbckr commented 2 years ago

@sergeyshaykhullin #5 fixed the issue for us. Thanks again!

rowi1de commented 1 year ago

Hey @sergeyshaykhullin did you experience any issue with 0.4.0 so far? https://github.com/appany/helm-oci-chart-releaser/issues/7