Skip to content

sql: add logging for grant_role#78653

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
RichardJCai:add_logging_for_grant_role
Mar 29, 2022
Merged

sql: add logging for grant_role#78653
craig[bot] merged 1 commit intocockroachdb:masterfrom
RichardJCai:add_logging_for_grant_role

Conversation

@RichardJCai
Copy link
Copy Markdown
Contributor

@RichardJCai RichardJCai commented Mar 28, 2022

GRANT ROLE now logs to the logpb.Channel_USER_ADMIN channel.
Example: GRANT role1, role2 TO role3, role4 logs
"RoleNames":"‹role1›",‹role2›","Members":"‹role3›",‹ role4›"

Release note: None

Fixes #78245

@RichardJCai RichardJCai requested a review from a team March 28, 2022 18:57
@RichardJCai RichardJCai requested a review from a team as a code owner March 28, 2022 18:57
@RichardJCai RichardJCai requested a review from a team March 28, 2022 18:57
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown
Contributor

@ajwerner ajwerner left a comment

Choose a reason for hiding this comment

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

:lgtm: mod the generated file

Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained

@rafiss
Copy link
Copy Markdown
Collaborator

rafiss commented Mar 28, 2022

This would have made the investigation in https://github.com/cockroachlabs/support/issues/1463 easier so I'm also in favor of a backport to 22.1. It's a new proto, so I don't know if 21.2 is eligible.

GRANT ROLE now logs to the logpb.Channel_USER_ADMIN channel.
Example: GRANT role1, role2 TO role3, role4 logs
"GranteeRoles":["‹role1›","‹role2›"],"Members":["‹role3›","‹ role4›"]

Release note: None
@RichardJCai RichardJCai force-pushed the add_logging_for_grant_role branch from 4a927db to 909368d Compare March 29, 2022 17:12
@RichardJCai
Copy link
Copy Markdown
Contributor Author

bors r+

@craig craig bot merged commit 897c2da into cockroachdb:master Mar 29, 2022
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 29, 2022

Build succeeded:

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.

log: Add log message for granting role to user.

4 participants