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

Removed extra tab in yaml example #672

Merged
merged 1 commit into from
Apr 25, 2024
Merged

Conversation

dd-georgiev
Copy link
Contributor

There were two extra spaces in the example for values.yaml when using existing configmap. This was resulting in the following error:

Error: UPGRADE FAILED: template: alloy/templates/controllers/daemonset.yaml:20:8: executing "alloy/templates/controllers/daemonset.yaml" at <include "alloy.pod-template" .>: error calling include: template: alloy/templates/controllers/_pod.yaml:2:42: executing "alloy.pod-template" at <.Values.alloy>: wrong type for value; expected map[string]interface {}; got interface {}

PR Description

Which issue(s) this PR fixes

Notes to the Reviewer

PR Checklist

  • CHANGELOG.md updated
  • Documentation added
  • Tests updated
  • Config converters updated

There were two extra spaces in the example for values.yaml  when using existing configmap. This was resulting in the following error:
```
Error: UPGRADE FAILED: template: alloy/templates/controllers/daemonset.yaml:20:8: executing "alloy/templates/controllers/daemonset.yaml" at <include "alloy.pod-template" .>: error calling include: template: alloy/templates/controllers/_pod.yaml:2:42: executing "alloy.pod-template" at <.Values.alloy>: wrong type for value; expected map[string]interface {}; got interface {}
```
@CLAassistant
Copy link

CLAassistant commented Apr 25, 2024

CLA assistant check
All committers have signed the CLA.

@tpaschalis tpaschalis merged commit 6947bad into grafana:main Apr 25, 2024
11 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 25, 2024
tpaschalis pushed a commit that referenced this pull request Apr 25, 2024
(cherry picked from commit 6947bad)

Co-authored-by: dd-georgiev <[email protected]>
@clayton-cornell clayton-cornell added the type/docs Docs Squad label across all Grafana Labs repos label Apr 25, 2024
hainenber pushed a commit to hainenber/alloy that referenced this pull request May 1, 2024
@rfratto rfratto added the backport-to-agent:no PR should NOT be backported to the agent repo. label May 3, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport release/v1.0 backport-to-agent:no PR should NOT be backported to the agent repo. frozen-due-to-age type/docs Docs Squad label across all Grafana Labs repos
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants