-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
sig-auth release notes: add advanced audit to action required, add selfsubjectrulesreview #412
Conversation
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.
minor change, otherwise lgtm.
release-1.8/release_notes_draft.md
Outdated
`v1beta1` event format, the log file defaults to JSON encoding, and the | ||
`--audit-policy-file` requires `kind` and `apiVersion` fields specifying | ||
what format version the `Policy` is using. | ||
|
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.
Advanced auditing has graduated from v1alpha1
to v1beta1
with the following changes to the default behavior:
- Webhook and the log file now output the
v1beta1
event format - The log file defaults to JSON encoding
- The
--audit-policy-file
requireskind
andapiVersion
fields specifying what format and version thePolicy
is using
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, thanks
…lfsubjectrulesreview
46f92b6
to
590fa4d
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
Update the file link path
Add notes about advanced auditing changes. cc @sttts @CaoShuFeng @soltysh
kubernetes/kubernetes#48051 has been approved, so adding it here.
/cc @radhikapc