Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Default values in Helm chart are broken in v5.5.0 #1315

Closed
yafanasiev opened this issue Nov 22, 2023 · 5 comments · Fixed by #1316
Closed

[Bug] Default values in Helm chart are broken in v5.5.0 #1315

yafanasiev opened this issue Nov 22, 2023 · 5 comments · Fixed by #1316
Labels
bug Something isn't working needs triage Indicates an issue or PR lacks a `triage/foo` label and requires one.

Comments

@yafanasiev
Copy link

Describe the bug

Helm chart installation fails

Error: YAML parse error on grafana-operator/templates/deployment.yaml: error converting YAML to JSON: yaml: line 36: did not find expected '-' indicator
helm.go:84: [debug] error converting YAML to JSON: yaml: line 36: did not find expected '-' indicator

Version
v5.5.0

To Reproduce

  1. Clone grafana-operator repository (or pull the chart using helm pull)
  2. Run helm template .
  3. See the error.

Expected behavior
Chart is templated using default values.

Suspect component/Location where the bug might be occurring
It was introduced in 3166d43. In this PR a duplicate env root key in values.yaml was introduced.

Runtime (please complete the following information):

  • OS: macOS 14.0 (23A344)
  • Grafana Operator Version v5.5.0
  • Helm version.BuildInfo{Version:"v3.13.2", GitCommit:"2a2fb3b98829f1e0be6fb18af2f6599e0f4e8243", GitTreeState:"clean", GoVersion:"go1.21.4"}

Additional context
Working workaround:

env:
  grafanaImage: ~
@yafanasiev yafanasiev added bug Something isn't working needs triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Nov 22, 2023
@yafanasiev
Copy link
Author

Maybe it would be worth introducing some kind of automated checks for linting Helm chart? I could help with that.

tamcore added a commit to gitgrave/grafana-operator that referenced this issue Nov 22, 2023
@NissesSenap
Copy link
Collaborator

@yafanasiev yes I was thinking about that as well. Sadly I haven't had the time. If you do I would love a separate PR for it.
I will fix a new release today after I looked at the PR

@NissesSenap
Copy link
Collaborator

@yafanasiev yes I was thinking about a hem template check as well. Sadly I haven't had the time. If you do I would love a separate PR for it.

@weisdd
Copy link
Collaborator

weisdd commented Nov 24, 2023

@yafanasiev FYI: helm lint and helm template were added in #1318 by @NissesSenap, so we should not hit the same issue ever again :)

@yafanasiev
Copy link
Author

@weisdd thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs triage Indicates an issue or PR lacks a `triage/foo` label and requires one.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants