install the postfix package in recipe ::postfix #19

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@anujbiyani
Contributor

anujbiyani commented Jan 29, 2013

I ran the monit::postfix recipe and noticed the monitoring wasn't working because postfix wasn't installed, so I've added it to the recipe.

I also added tests with ChefSpec; it's a cool RSpec-based unit-testing framework for cookbooks.

@apsoto

This comment has been minimized.

Show comment Hide comment
@apsoto

apsoto May 1, 2013

Owner

the postfix and ssh recipes are bit of a legacy of the extraction from my projects. I'm not sure whether monit should be the one providing package specific recipes going forward. I'm thinking monit provide the constructs to let the individual packages provide monit support. However, I'd like user feedback before making any decisions either way.

Owner

apsoto commented May 1, 2013

the postfix and ssh recipes are bit of a legacy of the extraction from my projects. I'm not sure whether monit should be the one providing package specific recipes going forward. I'm thinking monit provide the constructs to let the individual packages provide monit support. However, I'd like user feedback before making any decisions either way.

@apsoto apsoto closed this May 1, 2013

@anujbiyani

This comment has been minimized.

Show comment Hide comment
@anujbiyani

anujbiyani May 20, 2013

Contributor

I think in the case of postfix, since monit uses it to send emails, it'd be nice to fully include it in the cookbook. In general it makes sense that the monit cookbook does not install any packages, but installing postfix is a dependency if the user chooses to send emails via monit.

Contributor

anujbiyani commented May 20, 2013

I think in the case of postfix, since monit uses it to send emails, it'd be nice to fully include it in the cookbook. In general it makes sense that the monit cookbook does not install any packages, but installing postfix is a dependency if the user chooses to send emails via monit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment