-
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
Fix docs for advanced audit #45884
Fix docs for advanced audit #45884
Conversation
Hi @CaoShuFeng. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with 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. I understand the commands that are listed here. |
@@ -48,7 +48,6 @@ type Event struct { | |||
// Time the request reached the apiserver. | |||
Timestamp metav1.Time | |||
// Unique audit ID, generated for each request. | |||
// +optional |
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.
UID is important for identifying a single log. So I think it is not optional.
@timstclair |
@k8s-bot ok to test |
Thanks! /lgtm |
/cc @sttts |
// The user groups this rule applies to. If a user is considered matching | ||
// if the are a member of any of these groups | ||
// The user groups this rule applies to. A user is considered matching | ||
// if it has any member of the UserGroups |
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 it is a member of any of the UserGroups?
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.
Done.
1f89a40
to
22ba5eb
Compare
/lgtm Manually approving docs changes. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: CaoShuFeng, timstclair Assign the PR to them by writing
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
@k8s-bot gce etcd3 e2e test this |
1 similar comment
@k8s-bot gce etcd3 e2e test this |
Automatic merge from submit-queue (batch tested with PRs 45884, 45879, 45912, 45444, 45874) |
This change updates docs from this pr: #45862
Release note: