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

merge staging-v23.1.19 to release-23.1: released CockroachDB version 23.1.19. Next version: 23.1.20 #122635

Conversation

cockroach-teamcity
Copy link
Member

Release note: None
Epic: None
Release justification: non-production (release infra) change.

Santamaura and others added 5 commits March 8, 2023 17:51
This PR introduces the cluster setting
sql.auth.modify_cluster_setting_applies_to_all.enabled which determines
whether users with MODIFYCLUSTERSETTING are able to edit non
sql.defaults settings. It also allows this setting to prevent viewing
sql.default settings. Additionally, a deprecation warning is added for
this cluster setting to inform that it should be replaced once the
cluster is finalized

Fixes: cockroachdb#122023

Release note (bug fix): reintroduce
sql.auth.modify_cluster_setting_applies_to_all.enabled so that mixed
version clusters can migrate off this setting.
…1.19

Release note: None
Epic: None
Release justification: non-production (release infra) change.
…ort-staging-v23.1.19-122055

staging-v23.1.19: release-23.1: sql: regulate MODIFYCLUSTERSETTING with cluster setting
…ort-staging-v23.1.19-122234

staging-v23.1.19: release-23.1: released CockroachDB version 23.1.18. Next version: 23.1.19
…23.1.19. Next version: 23.1.20

Release note: None
Epic: None
Release justification: non-production (release infra) change.
@cockroach-teamcity
Copy link
Member Author

This change is Reviewable

Copy link
Contributor

@rsevinsky-cr rsevinsky-cr left a comment

Choose a reason for hiding this comment

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

Reviewed 7 of 7 files at r1, 1 of 1 files at r2.
Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @cockroach-teamcity)

Copy link
Contributor

@rsevinsky-cr rsevinsky-cr left a comment

Choose a reason for hiding this comment

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

Reviewed 8 of 8 files at r3, all commit messages.
Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @cockroach-teamcity)

@rsevinsky-cr rsevinsky-cr merged commit 827b9d2 into cockroachdb:release-23.1 Apr 18, 2024
6 checks passed
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.

None yet

5 participants