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
feat(helm): enable NodePort customization #6770
Conversation
app/kumactl/cmd/install/testdata/install-control-plane.dump-values.yaml
Outdated
Show resolved
Hide resolved
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.
Thanks @mmorel-35 ! Can we introduce just NodePort customization initially in this PR? I think typically it's not necessary to customize the port
s of Services since there's a new ClusterIP allocated for every Service but we can introduce if it's requested later.
app/kumactl/cmd/install/testdata/install-control-plane.dump-values.yaml
Outdated
Show resolved
Hide resolved
app/kumactl/cmd/install/testdata/install-control-plane.dump-values.yaml
Outdated
Show resolved
Hide resolved
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
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.
LGTM
Thanks @mmorel-35 !! |
Checklist prior to review
syscall.Mkfifo
have equivalent implementation on the other OS --UPGRADE.md
? --> Changelog:
entry here or add aci/
label to run fewer/more tests?