-
Notifications
You must be signed in to change notification settings - Fork 327
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
kuma-cp: add k8s Admission Web Hook to apply defaults to Mesh resources #297
Conversation
bd5c521
to
eb63537
Compare
xdsServer: | ||
grpcPort: 0 | ||
diagnosticsPort: %d | ||
diagnosticsPort: %%d |
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.
How does it work? What does double % mean?
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.
%
escapes another %
, so %%d
will become %d
after passing thorugh fmt.Sprintf
address: | ||
# Port the Admission WebHook Server should be listening on | ||
port: 5443 | ||
# Directory with a TLS cert and private key for the Admission WebHook Server |
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.
How about keeping comments the same as in code?
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.
fixed
eb63537
to
8b14573
Compare
60306dd
to
7ca9354
Compare
1aaa16d
to
f5fb69f
Compare
changes:
k8s Admission WebHook
to apply defaults toMesh
resources, such asbuilt-in CA
k8s Admission WebHook Server
usingRuntimePlugin
kumactl install control-plane
respectively