-
Notifications
You must be signed in to change notification settings - Fork 38.8k
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
Use audit.k8s.io/v1
as default value of option --audit-webhook-vers…
#70476
Conversation
/ok-to-test I guess we don't need to include |
740bf83
to
d598b41
Compare
cc @liggitt |
@@ -66,7 +66,7 @@ func TestAddFlags(t *testing.T) { | |||
"--audit-log-truncate-enabled=true", | |||
"--audit-log-truncate-max-batch-size=45", | |||
"--audit-log-truncate-max-event-size=44", | |||
"--audit-log-version=audit.k8s.io/v1alpha1", |
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.
revert this, this is testing that explicitly setting the flag can override the default
@@ -80,7 +80,7 @@ func TestAddFlags(t *testing.T) { | |||
"--audit-webhook-truncate-max-batch-size=43", | |||
"--audit-webhook-truncate-max-event-size=42", | |||
"--audit-webhook-initial-backoff=2s", | |||
"--audit-webhook-version=audit.k8s.io/v1alpha1", |
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.
revert this, this is testing that explicitly setting the flag can override the default
one comment about the test, lgtm otherwise |
d598b41
to
8e772d7
Compare
…ion and --audit-log-version in release 1.13
8e772d7
to
9671a03
Compare
/retest |
/assign @deads2k |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: charrywanganthony, liggitt 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 |
/kind feature |
…1 which were introduced in kubernetes/kubernetes#70476 Signed-off-by: Sandor Szücs <sandor.szuecs@zalando.de>
…ion and --audit-log-version in release 1.13