Issue while downloading the chart by passing the full URL. about helm HOT 6 CLOSED

Thanks! We can definitely add a helm repo update command into the process. Do you want to contribute a fix?

Probably would need to do this at this line: https://github.com/deliverybot/helm/blob/master/index.js#L236

from helm.

Hi @colinjfw,
I tried adding the repo add , it is working fine but still issue with helm upgrade for helmv2.
The below command works locally but not with github action -
COMMAND -
helm upgrade validator6 charts***ose.com/aonity ***install ***atomic ***namespace=valid6 ***dry***run ***version=1.1.1 ***values=./values.yaml ***values=./genesis.yaml ***values=./values.yml

ERROR - failed to download "charts***ose.com/aonity" (hint: running `helm repo update` may help
-- ADDED for helm rep update too.
Any suggestions ??
temp repo - https://github.com/raj-shekhar1/helm-action-bot

from helm.

hey @colinjfw,
waiting for your suggestions for the resolution and move ahead with a PR 🤞
@JulienBreux

from helm.

@raj-shekhar1 I'd suggest opening a pull request with the forked repository, then we can discuss with a code diff which will help me understand the changes :)

from helm.

sure @colinjfw

from helm.

@colinjfw i work with @raj-shekhar1 and have opened a PR: #23 to add a repo parameter (also to specify chart version - which did not seem to be happening). It needs some further testing (and tests written to handle these scenarios - i am not there yet)

interestingly enough even after having the repo added (either in the docker file, helm repo add or via the repo flag) helm2 still fails with:

ERROR - failed to download "..." (hint: running `helm repo update` may help

It seems that this is actually a misleading error message and is thrown because helm cannot read the repositories file (i'm not sure how this doesn't bubble up - and is an issue for the helm devs), As such it needed to be specified via the --home argument to the location docker installs the helm config (explicitly setting $HELM_HOME failed as helm update seems to ignore it).

I am opening this for your review :)

from helm.