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 type & provider for managing plugins #641

Merged
merged 1 commit into from Jan 27, 2015
Merged

Conversation

dveeden
Copy link
Contributor

@dveeden dveeden commented Jan 19, 2015

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.

end

it 'should find the plugin' do
shell("select plugin_name from information_schema.plugins where plugin_name='auth_socket'") do |r|
Copy link
Contributor

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

Copy link
Contributor

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.

Copy link
Contributor Author

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.

@cmurphy
Copy link
Contributor

cmurphy commented Jan 22, 2015

Could you add a section for this to the README?

@dveeden
Copy link
Contributor Author

dveeden commented Jan 24, 2015

I've added a section to the README

igalic added a commit that referenced this pull request Jan 27, 2015
Add type & provider for managing plugins
@igalic igalic merged commit 8dbe7c0 into puppetlabs:master Jan 27, 2015
@igalic
Copy link
Contributor

igalic commented Jan 27, 2015

thaaaaaaaaanks @dveeden!

cmurphy pushed a commit to cmurphy/puppetlabs-mysql that referenced this pull request Jan 27, 2015
underscorgan pushed a commit that referenced this pull request Jan 27, 2015
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.

None yet

4 participants