-
Notifications
You must be signed in to change notification settings - Fork 794
Add type & provider for managing plugins #641
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
Conversation
end | ||
|
||
it 'should find the plugin' do | ||
shell("select plugin_name from information_schema.plugins where plugin_name='auth_socket'") do |r| |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this would work, since shell() shells out to the sh shell, not the mysql shell. Should look more like https://github.com/puppetlabs/puppetlabs-mysql/blob/master/spec/acceptance/types/mysql_user_spec.rb#L27
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The shell()
command will actually call out to a sh
, so you would have to call shell('mysql -e "select plugin_name ...')
or something to get the mysql output.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. I've updated that line.
Could you add a section for this to the README? |
I've added a section to the README |
Add type & provider for managing plugins
thaaaaaaaaanks @dveeden! |
Fix acceptance tests from #641
The basics for working with MySQL plugins works.
Some plugins like auth_socket and validate_password are shipped with the server. The enterprise edition also has a auditlog, encryption and threadpool plugin. There are also third-party plugins which add things like JSON parsing, etc.