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

No reasons to manage separate files since confd_purge is available #812

Closed
wants to merge 1 commit into from
Closed

No reasons to manage separate files since confd_purge is available #812

wants to merge 1 commit into from

Conversation

vchepkov
Copy link
Contributor

@vchepkov vchepkov commented Jun 3, 2016

Module also prevents managing content of default files
if configuration files installed by rpm just removed, without supplying a sanitized safe content, they will be re-installed during rpm update,
this will leave site broken until next puppet run

Module also prevents managing content of default files
if configuration files installed by rpm just removed, without supplying a sanitized safe content, they will be re-installed during rpm update,
this will leave site broken until next puppet run
@wyardley
Copy link
Collaborator

wyardley commented Oct 6, 2016

#900 is this, but with the tests for those resources being absent removed, should address the Travis failures.

wyardley added a commit that referenced this pull request Oct 6, 2016
update of #812 (No reasons to manage separate files since confd_purge is available)
@wyardley wyardley closed this Oct 6, 2016
@wyardley
Copy link
Collaborator

wyardley commented Oct 6, 2016

Merged #900

darken99 pushed a commit to darken99/puppet-nginx that referenced this pull request Oct 13, 2016
cegeka-jenkins pushed a commit to cegeka/puppet-nginx that referenced this pull request Sep 13, 2019
Rubueno pushed a commit to Rubueno/puppet-nginx that referenced this pull request Oct 19, 2020
Rubueno pushed a commit to Rubueno/puppet-nginx that referenced this pull request Oct 19, 2020
update of voxpupuli#812 (No reasons to manage separate files since confd_purge is available)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants