What does this MR do?
Current migration code requires MySQL user to have admin rights on the database. The change uses MySQL's
information_schema tables rather than the restricted
Are there points in the code the reviewer needs to double check?
information_schema is scoped for the user by the looks of it. I have remove the extra scoping and compressed the PostgreSQL grantee into the main method. Not sure if this will work.
I will clean up the unit tests if this change is OK, wanted to get some feedback first.
I also wonder if the unit tests should/could be run with the least privileges needed as I imagine that they are currently running as an admin.
Why was this MR needed?
To ensure that the GitLab doesn't require MySQL admin rights.
Screenshots (if relevant)
Does this MR meet the acceptance criteria?
Changelog entry added, if necessary
API support added
Tests added for this feature/bug
Has been reviewed by UX
Has been reviewed by Frontend
Has been reviewed by Backend
Has been reviewed by Database
Conform by the merge request performance guides
Conform by the style guides
Squashed related commits together