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

Additional configuration options as Chef attributes #3

Closed
wants to merge 2 commits into from
Closed

Additional configuration options as Chef attributes #3

wants to merge 2 commits into from

Conversation

@organicveggie
Copy link
Contributor

@organicveggie organicveggie commented Mar 22, 2012

This extracts a bunch of the configuration options in monitrc into Chef attributes that can be overridden.

@apsoto
Copy link
Owner

@apsoto apsoto commented Mar 22, 2012

Re-adding my comments to the pull request:

line 23 of monitrc.erb looks like it doesn't have a closing %> for the port, line 27 has a > instead of a ? after the .nil check.

Can you verify these changes work (You've deployed it to a server?) and re-submit.

also could you switch to the new style of attribute reference (Ex: node.httpd.enable)

thanks

@apsoto apsoto closed this Mar 22, 2012
@organicveggie
Copy link
Contributor Author

@organicveggie organicveggie commented Mar 22, 2012

Story of my life. My cookbook repository has a submodule for monit that was pointing at your repo, instead of mine. No wonder it worked on the first try. :)

I'll resubmit when it's ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants