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

Add support for Postfix integration #167

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
3 participants
@phlipper
Copy link
Contributor

phlipper commented Dec 20, 2014

This PR introduces support for the Postfix integration.

@phlipper phlipper force-pushed the phlipper:postfix branch 4 times, most recently from 826f01c to a9f998c Dec 20, 2014

@miketheman miketheman added the feature label Dec 30, 2014

@phlipper phlipper force-pushed the phlipper:postfix branch from a9f998c to 1081142 Jan 7, 2015

@phlipper phlipper force-pushed the phlipper:postfix branch from 1081142 to bf04e17 Feb 5, 2015

@coveralls

This comment has been minimized.

Copy link

coveralls commented Feb 5, 2015

Coverage Status

Coverage remained the same at 100.0% when pulling bf04e17 on phlipper:postfix into b2824ae on DataDog:master.

1 similar comment
@coveralls

This comment has been minimized.

Copy link

coveralls commented Feb 5, 2015

Coverage Status

Coverage remained the same at 100.0% when pulling bf04e17 on phlipper:postfix into b2824ae on DataDog:master.

@phlipper phlipper force-pushed the phlipper:postfix branch from bf04e17 to d851bf9 Jun 29, 2015

@phlipper phlipper force-pushed the phlipper:postfix branch from d851bf9 to a695eff Jun 29, 2015

@phlipper

This comment has been minimized.

Copy link
Contributor Author

phlipper commented Jun 29, 2015

This has been rebased on top of the latest master, and contains only the postfix changes.

@miketheman miketheman added this to the Next minor milestone Jul 4, 2015

@miketheman

This comment has been minimized.

Copy link
Collaborator

miketheman commented Jul 4, 2015

@phlipper Thanks! I'd like to take one pass at seeing if we can get the template syntax down to the simpler, new template model: https://github.com/DataDog/chef-datadog/blob/master/templates/default/ntp.yaml.erb

This would allow for arbitrary extensions to the plugin to be used immediately by manipulating the attribute hash, instead of waiting for a cookbook release to the template.

@miketheman

This comment has been minimized.

Copy link
Collaborator

miketheman commented Oct 13, 2015

Merged, thanks for working on this!

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.