-
Notifications
You must be signed in to change notification settings - Fork 279
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
stable/node-local-dns Extend daemonset arg list #503
stable/node-local-dns Extend daemonset arg list #503
Conversation
stable/node-local-dns/values.yaml
Outdated
setupInterface: {} | ||
|
||
setupIptables: {} | ||
|
||
skipTeardown: {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think these are not maps, they are simply booleans, no? e.g. -setupiptables=false
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, they are boolean. Changed. Can you check please?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @max-rocket-internet. Any news?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The README hasn't been updated with the new boolean values
8f9770e
to
3b97be9
Compare
- "{{ include "node-local-dns.fullname" . }}-upstream" | ||
- -skipteardown={{ .Values.config.skipTeardown }} | ||
- -setupinterface={{ .Values.config.setupInterface }} | ||
- -setupiptables={{ .Values.config.setupIptables }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is change you are re-ordering all the args and it looks like the order matters, no?
Before: -localip 169.254.20.25,172.20.0.10 -conf /etc/Corefile -upstreamsvc node-local-dns-upstream
After: -localip -conf /etc/Corefile -upstreamsvc 169.254.20.25,172.20.0.10 node-local-dns-upstream...
55d438b
to
a39b00d
Compare
yeah, my bad.
Please take a look |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, nice, thanks @kkonnov
Description
Extend daemonset arg list
Checklist
[stable/mychartname]
)