Skip to content
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 proposal to rotate audit log #190

Merged
merged 1 commit into from
Apr 26, 2022

Conversation

stonezdj
Copy link
Contributor

Add options to forward audit_log to log stream
Add option to rotate audit log table

@stonezdj stonezdj force-pushed the audit_log_proposal branch 2 times, most recently from 29ed8d8 to 0d3b3aa Compare March 28, 2022 02:44
@wy65701436
Copy link
Contributor

wy65701436 commented Apr 7, 2022

link it: goharbor/harbor#128

@stonezdj stonezdj force-pushed the audit_log_proposal branch 2 times, most recently from df247db to 0961843 Compare April 12, 2022 02:30
   Add options to forward audit_log to log stream
   Add option to rotate audit log table

Signed-off-by: stonezdj <stonezdj@gmail.com>
Copy link
Contributor

@wy65701436 wy65701436 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Member

@chlins chlins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Contributor

@steven-zou steven-zou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@stonezdj stonezdj merged commit cbe574e into goharbor:main Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants