-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Add --version flag to podsecurity-webhook command #105749
Conversation
@tallclair: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
needs to allow this import to pass verify, lgtm otherwise |
57f8bea
to
38eed97
Compare
38eed97
to
0c19298
Compare
I decided to just allow all of component-base. |
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
It seems that we can use the flag as the version labels after this PR merged? : ) It's awesome ! |
cmdName := "podsecurity-webhook" | ||
if executable, err := os.Executable(); err == nil { | ||
cmdName = filepath.Base(executable) | ||
} |
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.
if there is err, then do we need to bail out ?
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.
Nah, the podsecurity-webhook
is the fallback... I don't actually know what situation this would fail in...
0c19298
to
c7cdf19
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jyz0309, liggitt, tallclair 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:
Add the
--version
flag to the webhook command. This will be important with the inclusion of version information in the metrics (#104217) to ensure we're properly including the version the webhook binary, and also be useful to users of the binary to know what version they have./sig auth