appany / helm-oci-chart-releaser

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

appany/helm-oci-chart-releaser:0.4.0 Helm Push is broken (+ incorrect status) on ubuntu-20.04 #7

Open rowi1de opened 1 year ago

rowi1de commented 1 year ago

Describe the bug

To Reproduce

From the logs you see that the helm push is not finding the output:

Run echo *** | helm registry login -u __token__ --password-stdin ghcr.io/xxx
Login succeeded
  helm package k8s/helm-app --version 0.0.123
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    HELM_EXPERIMENTAL_OCI: 1
Successfully packaged chart and saved it to: 

saved it to: is literally empty

The following helm push is assuming the previous step was successful:

  helm push service-app-0.0.123.tgz oci://ghcr.io/xxx/yyy
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    HELM_EXPERIMENTAL_OCI: 1
The Kubernetes package manager

To begin working with Helm, run the 'helm init' command:

    $ helm init

This will set up any necessary local configuration.

It will just print the helm help, because the packaged file could not be found

sergeyshaykhullin commented 1 year ago

I can't reproduce this on v0.4.0, does your directory k8s/helm-app contains service-app?

  - name: Chart
    uses: appany/helm-oci-chart-releaser@v0.4.0
    with:
      name: ${{ ... }}
      repository: ${{ ... }}
      tag: ${{ steps.release.outputs.tag }}
      path: src
      registry: ${{ ... }}
      registry_username: ${{ ... }}
      registry_password: ${{ ... }}

image image

rowi1de commented 1 year ago

@sergeyshaykhullin thanks for looking into it. Sorry the bug was on my end, we included the following action, which installed helm version 3.0.0.-alpha1 for some reasons

- name: Install helm
        uses: Azure/setup-helm@v1

However this action still reports success, even if nothing was pushed .. this could be a nice improvement

almereyda commented 1 year ago

Yes, introducing errors that would break a workflow when raised can help in finding issues for stabilising the pipelines.