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

mariadb_configuration template uses current cookbook as template source; documentation issue #66

Closed
mstyne opened this issue May 13, 2015 · 5 comments

Comments

@mstyne
Copy link
Contributor

mstyne commented May 13, 2015

When defining a mariadb_configuration resource, the template resource in configuration.rb references new_resource.cookbook, which uses the name of the calling cookbook, which typically(?) isn't maraidb, and by default, doesn't have the templates in use defined.

================================================================================
Error executing action `create` on resource 'template[/etc/mysql/conf.d/some_conf.cnf]'
================================================================================

Chef::Exceptions::FileNotFound
------------------------------
Cookbook 'some-cookbook' (2.0.0) does not contain a file at any of these locations:
        templates/debian-7.8/conf.d.generic.erb
        templates/debian/conf.d.generic.erb
        templates/default/conf.d.generic.erb
        templates/conf.d.generic.erb

Changing the value to:

cookbook 'mariadb'

resolves this issue. Additionally, the documentation referencing the option hash as:

  option {foo: 'bar'}

fails with a syntax error, while something like

option :foo => bar,
  :baz => quux,

works. If these are both considered legitimate bugs, I'd be happy to put together a pull request to resolve them.

@sinfomicien
Copy link
Contributor

Please submit a PR, seems you're right :)

@sinfomicien
Copy link
Contributor

I just corrected the mariadb_configuration problem. I let you submit a PR concerning the documentation :)

@mstyne
Copy link
Contributor Author

mstyne commented Jul 31, 2015

@sinfomicien Sure, I can put together a pull request for the documentation. Thanks :)

@sinfomicien
Copy link
Contributor

As the PR is merged, i close the issue. Thx for your help!

@lock
Copy link

lock bot commented Jul 24, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Jul 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants