Skip to content

[DPE-7322] Support predefined roles #635

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sinclert-canonical
Copy link
Contributor

This PR implements the set of predefined roles proposed in this specification.

Additional changes:

  • Implemented MySQL lib function to fully support the usage of extra-user-roles config argument.
  • Harmonized MySQL config arguments to all user underscore (_) instead of mixing dashes and underscores.

@sinclert-canonical sinclert-canonical added the enhancement New feature, UI change, or workload upgrade label Jul 22, 2025
@sinclert-canonical sinclert-canonical force-pushed the sinclert/7322/predefined-roles branch 4 times, most recently from 63092c7 to c8c1ee4 Compare July 28, 2025 09:10
@sinclert-canonical sinclert-canonical force-pushed the sinclert/7322/predefined-roles branch from 36e6244 to f827f3c Compare August 11, 2025 09:33
@sinclert-canonical sinclert-canonical force-pushed the sinclert/7322/predefined-roles branch from f827f3c to bee7876 Compare August 11, 2025 10:23
@@ -133,7 +134,7 @@ def wait_until_mysql_connection(self) -> None:
# Increment this major API version when introducing breaking changes
LIBAPI = 0

LIBPATCH = 89
LIBPATCH = 90
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We would need to bump this number whenever Manual Primary Switchover PR changes are ported to K8s.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature, UI change, or workload upgrade Libraries: Out of sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant