-
Notifications
You must be signed in to change notification settings - Fork 7
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
feat: add cluster audit logs view to kubernetes dashboard #1861
feat: add cluster audit logs view to kubernetes dashboard #1861
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.
has this been modified to take the gql fixes into account yet?
Yep, it's ready. I have updated the screenshot in the PR description. |
cell: ({ getValue }) => <UserInfo user={getValue() ?? {}} />, | ||
}) | ||
|
||
const columns = [userColumn, pathColumn, methodColumn, insertedAtColumn] |
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.
super NIT-y, but I think the order here should be [method, path, actor, timestamp]
7e0e0bb
to
27df631
Compare
Added audit logs table view to kubernetes dashboard.
![image](https://private-user-images.githubusercontent.com/2285385/411046195-7aa73969-4483-4d52-ad71-06e15a80812a.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk1MDM2MDQsIm5iZiI6MTczOTUwMzMwNCwicGF0aCI6Ii8yMjg1Mzg1LzQxMTA0NjE5NS03YWE3Mzk2OS00NDgzLTRkNTItYWQ3MS0wNmUxNWE4MDgxMmEucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI1MDIxNCUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNTAyMTRUMDMyMTQ0WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9YTZhNjVmMzU1YzBlYjM4YmRlMzU3ZWZlZGFmMGM4YzcyMzFkNmIyNmE2NDdhMTdjNjZlOTc4Y2U1MzhlNDRmNSZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QifQ.SmB0oBEKixkmtLptmMvM7E6Zmq_bJYOE9jW8yaP6dgw)
Test Plan
Locally w/ dev env
Checklist