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

Added consul-check-def provider #67

Merged
merged 1 commit into from Oct 29, 2014

Conversation

Projects
None yet
4 participants
@lyrixx
Copy link
Contributor

lyrixx commented Oct 28, 2014

refs #66

@coveralls

This comment has been minimized.

Copy link

coveralls commented Oct 28, 2014

Coverage Status

Coverage remained the same when pulling 1a2d5e5 on lyrixx:check-def into 1177d9b on johnbellone:master.

johnbellone added a commit that referenced this pull request Oct 29, 2014

Merge pull request #67 from lyrixx/check-def
Added consul-check-def provider

@johnbellone johnbellone merged commit 39940d0 into sous-chefs:master Oct 29, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

@lyrixx lyrixx deleted the lyrixx:check-def branch Oct 29, 2014


action :create do
file new_resource.path do
user node['consul']['service_user']

This comment has been minimized.

Copy link
@inean

inean Nov 18, 2014

Contributor

If consul service is configured to be 'init' based, recipe won't create node['consul']['service_user'] and this will fail (for init case, we should use 'root' user and group instead). @lyrixx

This comment has been minimized.

Copy link
@lyrixx

lyrixx Nov 18, 2014

Author Contributor

Can you open an issue for that ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.