-
Notifications
You must be signed in to change notification settings - Fork 82
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
Enhance audit log #253
base: main
Are you sure you want to change the base?
Enhance audit log #253
Conversation
This proposal not only introduces a new auditlogo but also add much more events.
I suggest also making the auditlog plugable, meaning that in the future it would be possible to add other types of backends apart from postgres, like opentelemtry or loki.. I am looking into few options and Unlogged tables directly or in combination with stored procedures can provide a huge improvement in write performance. |
Previous audit log has the log forward option, it can avoid log in the database |
The problem with the existing approach is that when this is enabled. Audit logs aren't displayed in Harbor anymore. All events are forwarded to syslog. What I had in mind was had the option to implement a different storage backend instead of the postgres. |
When you configured forward log to LogInsight or ELK, you should search the audit log in the ELK's query interface. |
@Vad1mo This is essentially a query module in Harbor that works with different backends. |
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
Signed-off-by: stonezdj <[email protected]>
cb6b70f
to
abd87bb
Compare
Replace postgres is a big task, we are focusing on urgent requirements, have no bandwidth to handle so far. |
No description provided.