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.

Sign up for GitHub

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

Jump to bottom

Missing quote #13

Closed
Fissium opened this issue Nov 15, 2024 · 0 comments · Fixed by #14
Closed

Missing quote #13

Fissium opened this issue Nov 15, 2024 · 0 comments · Fixed by #14
Assignees
@ialejandro @amartingarcia
Labels
bug Something isn't working

Comments

@Fissium
Copy link

Fissium commented Nov 15, 2024 •
edited
Loading

Describe the bug a clear and concise description of what the bug is.

helm-cruise-control/charts/cruise-control/values.yaml

Line 201 in aaed32d

"NW_IN": 1000",

What's your helm version?

3.16.3

What's your kubectl version?

1.31.2

Chart version

1.1.4

Explain bug

The configuration contains a missing opening quote for the value "NW_IN": 1000", leading to a syntax error.

What you expected to happen?

No response

How to reproduce it?

No response

Enter the changed values of values.yaml?

No response

Enter the command that you execute when chart failing/misfunctioning

I use argocd

More info

No response
@Fissium Fissium added the bug Something isn't working label Nov 15, 2024
ialejandro added a commit that referenced this issue Nov 15, 2024
@ialejandro
fix: probes and quotes
496fae6 
* Fix typo on NW_INT default value
* Improve probes

Closes #12
Closes #13
@ialejandro ialejandro mentioned this issue Nov 15, 2024
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ialejandro ialejandro

@amartingarcia amartingarcia

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: probes and quotes devops-ia/helm-cruise-control
3 participants
@ialejandro @Fissium @amartingarcia