Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Add note about targeting a specific Helm version for the Helm operator #2830

Merged
merged 1 commit into from
Feb 12, 2020

Conversation

edwardyoung
Copy link

Update "Get started with Flux using Helm" tutorial to include an example of installing the Helm operator for users using Helm v3. The current example installs the Helm operator for Helm v2 causing the helm-operator pod to crash with the following error if you are using Helm v3:

ts=2020-02-06T21:04:41.725026537Z caller=helm.go:59 component=helm version=v2 error="error creating Client (v2) client: services "tiller-deploy" not found"

Copy link
Member

@hiddeco hiddeco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of making it this verbose, I would simply mention that a specific Helm version can be targeted by setting the helm.versions value, and that the default is to enable both Helm v2 and v3.

Copy link
Member

@hiddeco hiddeco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you @edwardyoung 💯

@hiddeco hiddeco added the docs Issue or PR relates to documentation label Feb 10, 2020
@hiddeco hiddeco added this to the 1.19.0 milestone Feb 10, 2020
@hiddeco hiddeco changed the title Add helm-operator install example for Helm v3 Add note about targeting a specific Helm version for the Helm operator Feb 10, 2020
@hiddeco hiddeco merged commit a9e821b into fluxcd:master Feb 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
docs Issue or PR relates to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants