Azure / setup-helm

Github Action for installing Helm
MIT License
141 stars 48 forks source link

Bug: Latest version is grabbing alpha version of tool #97

Closed KenBurk closed 2 years ago

KenBurk commented 2 years ago

What happened?

Expecting: Helm tool version: 'v3.8.2' has been cached at /opt/hostedtoolcache/helm/3.8.2/x64/linux-amd64/helm actual Helm tool version: 'v3.0.0-alpha.1' has been cached at /opt/hostedtoolcache/helm/3.0.0-alpha.1/x64/linux-amd64/helm

Version

Runner

self-hosted

Relevant log output

Run azure/setup-helm@v1 with: version: latest ... creating: linux-amd64/ inflating: linux-amd64/helm
inflating: linux-amd64/LICENSE
inflating: linux-amd64/README.md

kxiao-msk commented 2 years ago

we face the same issue, can this be fixed ASAP?

ShiroTian commented 2 years ago

me too

umutkacar commented 2 years ago

We were using azure/setup-helm@18bc76811624f360dbd7f18c2d4ecb32c7b87bab # v1 and we also encountered this issue.

It was installing an alpha version: Helm tool version: 'v3.0.0-alpha.1' has been cached at /opt/hostedtoolcache/helm/3.0.0-alpha.1/x64/linux-amd64/helm and it was complaining at the deploy step by throwing: Error: unknown flag: --atomic

Switching to azure/setup-helm@20d2b4f98d41febe2bbca46408499dbb535b6258 # v3.0 seems to have fixed the issue.

I need to observe a little more to confirm.

OliverMKing commented 2 years ago

This will be fixed in #100. I'm going to roll it out as soon as possible. #98 and #99 are duplicates of this.

OliverMKing commented 2 years ago

This has been fixed and released in v3.4 (and v3). Please upgrade to one of these versions. Thanks!