-
Notifications
You must be signed in to change notification settings - Fork 7.5k
--repo option is using local folder as chart instead of looking for the chart inside the provided repository #11141
Copy link
Copy link
Closed
Labels
bugCategorizes issue or PR as related to a bug.Categorizes issue or PR as related to a bug.in progress
Description
Output of helm version:
version.BuildInfo{Version:"v3.9.0", GitCommit:"7ceeda6c585217a19a1131663d8cd1f7d641b2a7", GitTreeState:"clean", GoVersion:"go1.17.5"}
Output of kubectl version: `
Client Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.2", GitCommit:"f66044f4361b9f1f96f0053dd46cb7dce5e990a8", GitTreeState:"clean", BuildDate:"2022-06-15T14:22:29Z", GoVersion:"go1.18.3", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.4
Server Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.1", GitCommit:"3ddd0f45aa91e2f30c70734b175631bec5b5825a", GitTreeState:"clean", BuildDate:"2022-05-24T12:18:48Z", GoVersion:"go1.18.2", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.): minikube version: v1.26.0
Potential bug found
When I use the option --repo, Helm tries to use the local folder as chart instead of getting the chart from the repository.
$ ls
redis
$ helm repo list
Error: no repositories to show
$ helm upgrade -i --create-namespace --namespace test redis-test --repo https://charts.bitnami.com/bitnami redis --debug
history.go:56: [debug] getting history for release redis-test
Release "redis-test" does not exist. Installing it now.
install.go:178: [debug] Original chart version: ""
install.go:195: [debug] CHART PATH: /home/rafaelromcar/Tech/test/kube/tech-stack/redis
Error: Chart.yaml file is missing
helm.go:84: [debug] Chart.yaml file is missing
helm.sh/helm/v3/pkg/chart/loader.LoadFiles
helm.sh/helm/v3/pkg/chart/loader/load.go:158
helm.sh/helm/v3/pkg/chart/loader.LoadDir
helm.sh/helm/v3/pkg/chart/loader/directory.go:119
helm.sh/helm/v3/pkg/chart/loader.DirLoader.Load
helm.sh/helm/v3/pkg/chart/loader/directory.go:41
helm.sh/helm/v3/pkg/chart/loader.Load
helm.sh/helm/v3/pkg/chart/loader/load.go:62
main.runInstall
helm.sh/helm/v3/cmd/helm/install.go:204
main.newUpgradeCmd.func2
helm.sh/helm/v3/cmd/helm/upgrade.go:121
github.com/spf13/cobra.(*Command).execute
github.com/spf13/cobra@v1.4.0/command.go:856
github.com/spf13/cobra.(*Command).ExecuteC
github.com/spf13/cobra@v1.4.0/command.go:974
github.com/spf13/cobra.(*Command).Execute
github.com/spf13/cobra@v1.4.0/command.go:902
main.main
helm.sh/helm/v3/cmd/helm/helm.go:83
runtime.main
runtime/proc.go:255
runtime.goexit
runtime/asm_amd64.s:1581
If I go inside the folder redis it works:
$ helm upgrade -i --create-namespace --namespace test redis-test --repo https://charts.bitnami.com/bitnami redis
Release "redis-test" does not exist. Installing it now.
NAME: redis-test
LAST DEPLOYED: Wed Jul 13 10:01:46 2022
NAMESPACE: test
STATUS: deployed
REVISION: 1
TEST SUITE: None
NOTES:
CHART NAME: redis
CHART VERSION: 16.13.2
APP VERSION: 6.2.7
** Please be patient while the chart is being deployed **
[more ouput here]
If --repo is used, the chart should be searched only inside the provided repository and never locally IMHO.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugCategorizes issue or PR as related to a bug.Categorizes issue or PR as related to a bug.in progress