profile
viewpoint

Ask questionsBug: `--repo` should takes precedence over local folder

Using local chart when a --repo flag is provided doesn't make sense, the assumption in LocateChart function is not correct in this case,

Is it possible to ignore the local folders when --repo is provided?

Output of helm version:

version.BuildInfo{Version:"v3.1.2", GitCommit:"d878d4d45863e42fd5cff6743294a11d28a9abce", GitTreeState:"clean", GoVersion:"go1.13.8"}
helm/helm

Answer questions technosophos

On what grounds are you calling this a bug, @mattfarina? If this was explicitly a design choice made years ago. Looking at the repo, both Helm2 and Helm3 have always behaved this way. Changing it now constitutes a breaking change, since the behavior users have come to expect will change drastically (a completely different chart can be fetched).

If you want to call it a bug, you need to justify why it is a bug, not that it is "better UX". Better UX can be deferred until Helm 4. By my reading, changing this behavior violates the first bullet point on the braking changes list: https://github.com/helm/helm/blob/master/CONTRIBUTING.md#semantic-versioning

And thus I have played my role as the SemVer enforcer for the day.

useful!

Related questions

Error: open .helm\repository\cache\local-index.yaml hot 2
Helm upgrade fails the release after adding a new resource hot 2
Error: validation: chart.metadata is required when using --repo - helm hot 2
Error: could not find a ready tiller pod hot 2
Upgrading releases with immutable resources (e.g. persitentVolume) hot 2
"Error: Transport is closing" message when attempting to install hot 1
helm 3: Not able to push chart to docker hub hot 1
Helm3: No 'init', doesn't use existing ~/.helm hot 1
No kind Job is registered for version batch/v1 hot 1
error calling eq: invalid type for comparison hot 1
upgrade apiVersion in k8s from 1.14 to 1.16 FAILED hot 1
Helm v 2.9.1 Error: Can't get a valid version for repositories even when the chart is available - helm hot 1
Upgrades fails with: grpc: received message larger than max hot 1
Helm init fails on Kubernetes 1.16.0 hot 1
Helm error with forwarding ports hot 1
Github User Rank List