-
Notifications
You must be signed in to change notification settings - Fork 891
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
introduce --tls-min-version flag to karmada-webhook #1278
introduce --tls-min-version flag to karmada-webhook #1278
Conversation
f8c8d7e
to
fc698b9
Compare
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. Just proposed some comments on it.
6b75281
to
f23ea9d
Compare
kubectl logs info:
386
--skip_headers If true, avoid header prefixes in the log messages
387
--skip_log_headers If true, avoid headers when opening log files
388
--stderrthreshold severity logs at or above this threshold go to stderr (default 2)
389
--tls-min-version string Minimum TLS version supported. Possible values: VersionTLS10, VersionTLS11, VersionTLS12, VersionTLS13. (default "VersionTLS13")
390
-v, --v Level number for the log level verbosity (default 0)
391
--vmodule moduleSpec comma-separated list of pattern=N settings for file-filtered logging
392
393
ada-webhook [command] --help" for more information about a command.
394
395
invalid TLSMinVersion VersionTLS13: expects 1.0, 1.1, 1.2, 1.3 or empty |
f23ea9d
to
9441e9c
Compare
Sorry my bad. |
Co-authored-by: Hongcai Ren <[email protected]> Signed-off-by: carlory <[email protected]>
7f4eb96
to
403de9c
Compare
/cc @RainbowMango It's ready for review. |
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
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: RainbowMango The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Some users want to config the TLS MinVersion for karmada-webhook.
Which issue(s) this PR fixes:
Fixes #1272
Special notes for your reviewer:
Does this PR introduce a user-facing change?: