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

Add MariaDB user management #243

Merged
merged 1 commit into from
Oct 17, 2024
Merged

Add MariaDB user management #243

merged 1 commit into from
Oct 17, 2024

Conversation

Kidswiss
Copy link
Contributor

@Kidswiss Kidswiss commented Oct 14, 2024

Summary

  • Added MariaDB usermanagement analog to PostgreSQL usermanagement

Checklist

  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • Update tests.
  • Link this PR to related issues.

@Kidswiss Kidswiss added the minor label Oct 14, 2024
@Kidswiss Kidswiss marked this pull request as draft October 14, 2024 11:18
@Kidswiss Kidswiss force-pushed the add/mariadb_user_mgmt branch 8 times, most recently from 2fcdf8e to b4d2187 Compare October 15, 2024 09:03
@Kidswiss Kidswiss marked this pull request as ready for review October 15, 2024 09:17
@Kidswiss Kidswiss force-pushed the add/mariadb_user_mgmt branch 2 times, most recently from c4e44ee to f9e98f8 Compare October 15, 2024 11:22
@Kidswiss Kidswiss requested review from a team, TheBigLee, wejdross and zugao and removed request for a team October 15, 2024 11:24
It's now possible to do user management with MariaDB. The logic is
exactly the same as with PostgreSQL.
@Kidswiss Kidswiss force-pushed the add/mariadb_user_mgmt branch from 09a33f2 to 69fddb1 Compare October 17, 2024 11:12
@Kidswiss Kidswiss requested a review from wejdross October 17, 2024 11:12
Copy link
Collaborator

@zugao zugao left a comment

Choose a reason for hiding this comment

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

LGTM

@Kidswiss Kidswiss merged commit c433fcf into master Oct 17, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants