Skip to content

(MODULES-7857) Support user creation on galera#1130

Merged
david22swan merged 1 commit intopuppetlabs:masterfrom
MaxFedotov:fix_user_creation_on_galera
Oct 12, 2018
Merged

(MODULES-7857) Support user creation on galera#1130
david22swan merged 1 commit intopuppetlabs:masterfrom
MaxFedotov:fix_user_creation_on_galera

Conversation

@MaxFedotov
Copy link
Copy Markdown

Support user creation on galera. Add IF NOT EXISTS to CREATE USER in mysql_user provider
Description of issue in jira - https://tickets.puppetlabs.com/browse/MODULES-7857

@david22swan
Copy link
Copy Markdown
Member

@MaxFedotov Could you add some test coverage for your new feature and update the docs please?

@MaxFedotov
Copy link
Copy Markdown
Author

MaxFedotov commented Oct 11, 2018

@david22swan Hi. Can you please clarify, which docs should be updated? This PR doesn't add new functionality, it's just reuses the same logic, which is already implemented for DROP USER

self.class.mysql_caller("DROP USER #{if_exists}'#{merged_name}'", 'system')
. I've looked throught all docs, and wasn't able to find anything related to this.

@MaxFedotov MaxFedotov force-pushed the fix_user_creation_on_galera branch from 9ede9a8 to 784fc81 Compare October 11, 2018 19:56
@MaxFedotov
Copy link
Copy Markdown
Author

MaxFedotov commented Oct 11, 2018

@david22swan I've rebased on current master and slightly modified my PR. Also i've updated tests.

@MaxFedotov MaxFedotov force-pushed the fix_user_creation_on_galera branch from 784fc81 to 5a5789c Compare October 11, 2018 21:47
@MaxFedotov MaxFedotov force-pushed the fix_user_creation_on_galera branch from 5a5789c to 8811c22 Compare October 11, 2018 22:09
@david22swan david22swan merged commit 2444050 into puppetlabs:master Oct 12, 2018
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