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

Use the run_state instead of node #215

Closed
jswarren4 opened this issue Aug 5, 2014 · 3 comments
Closed

Use the run_state instead of node #215

jswarren4 opened this issue Aug 5, 2014 · 3 comments

Comments

@jswarren4
Copy link

Use run_state instead of node attributes to set passwords (e.g. node['mysql']['server_root_password']) as with sous-chefs/jenkins#158

@kramvan1
Copy link
Contributor

kramvan1 commented Aug 8, 2014

Sounds like a better approach, +1

@someara
Copy link
Contributor

someara commented Aug 25, 2014

Hi!

The mysql_service resource takes the server_root_password as a parameter. If you'd like to use run_state, please use the mysql_service resource directly in your recipe instead of include_recipe 'mysql::server'. I'll be removing the recipes and attributes directories in the 6.0 version of this cookbook. Closing for now!

-s

@someara someara closed this as completed Aug 25, 2014
openstack-gerrit pushed a commit to openstack-archive/cookbook-openstack-ops-database that referenced this issue Sep 18, 2014
Set the mysql root password directly on the server resource, so it
does not need to be an attribute.  This keeps passwords sourced from
encrypted data bags out of node attributes. Note that as indicated
in sous-chefs/mysql#215 all recipes
will be removed from the mysql cookbook in favor of having consumers
work with the resources directly, thereby providing more flexibility
in terms of how the various configuration settings are sourced.
Including the "server" recipe is not going to be a viable option
once that has happened, so this change not only addresses the
attribute issue, but it will also make upcoming changes to the
mysql cookbook more seamless.  Note that the reference to the
default_version_for function had to be removed, because it is no
longer in the cookbook and therefore the build failed. It is no
longer necessary to set the version, as described in the README.md
file:

When [the version parameter is] omitted, it will install the
default MySQL version for the target platform.

Note that the code that sets node attributes in
spec/mysql-server_spec.rb was removed because it was interfering
with the new test and it did not seem to serve any purpose--there
were no failures after it was removed.

blueprint no-secret-attributes

Change-Id: I72be1e2131a107ddc43754bc5cf40a4ce7face95
@lock
Copy link

lock bot commented Jun 2, 2019

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 Jun 2, 2019
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

3 participants