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

Cannot set multiple days in excluded Weekdays #136

Closed
nipuns11 opened this issue Aug 8, 2019 · 2 comments
Closed

Cannot set multiple days in excluded Weekdays #136

nipuns11 opened this issue Aug 8, 2019 · 2 comments
Labels

Comments

@nipuns11
Copy link

nipuns11 commented Aug 8, 2019

$ helm install stable/chaoskube --set dryRun=false --set namespaces=test --set interval=10m --set timezone=America/New_York --set excludedWeekdays="Mon,Wed,Thu,Fri" --set excludedTimesOfDay="08:00-18:00"

Error: failed parsing --set data: key "Wed" has no value (cannot end with ,)

Cannot set multiple days in excluded Weekdays

@linki
Copy link
Owner

linki commented Aug 8, 2019

@nipuns11 Yeah, that's a bit annoying. The reason is that --set can take multiple key/value pairs at once and in order to separate them it uses the comma. See that example for instance:

Multiple values are separated by , characters. So --set a=b,c=d becomes:

a: b
c: d

That makes the comma a reserved character within --set values so that you need to escape it.

The following will solve your issue (note the added \ before the , in excludedWeekdays):

helm install stable/chaoskube --set dryRun=false --set namespaces=test --set interval=10m --set excludedWeekdays="Mon\,Wed\,Thu\,Fri" --set excludedTimesOfDay="08:00-18:00"

@linki linki added the question label Aug 8, 2019
@nipuns11
Copy link
Author

nipuns11 commented Aug 8, 2019

Thanks :-)

@nipuns11 nipuns11 closed this as completed Aug 8, 2019
@linki linki pinned this issue Aug 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants