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

Ensure module packages are installed before evaluating conf.d dir #1378

Merged
merged 1 commit into from
Feb 22, 2016

Conversation

jasonhancock
Copy link
Contributor

The problem I'm running into on EL7 is that when the ssl module is enabled, the mod_ssl package gets installed and drops /etc/httpd/conf.d/ssl.conf. We want to remove that file, but the purge for /etc/httpd/conf.d is being evaluated before the mod_ssl package is being installed. Because both ports.conf and ssl.conf have Listen statements for port 443, apache won't start. On the second client run the ssl.conf file will be removed and apache starts fine.

This change ensures that the mod_ssl package gets installed prior to the purge on /etc/httpd/conf.d being evaluated so that the ssl.conf file gets removed on the first client run.

The problem I'm running into on EL7 is that when the ssl module is
enabled, the mod_ssl package gets installed and drops
`/etc/httpd/conf.d/ssl.conf`. We want to remove that file, but the purge
for `/etc/httpd/conf.d` is being evaluated before the mod_ssl package is
being installed. On the second client run the `ssl.conf` file will be
removed.

This change ensures that the `mod_ssl` package gets installed prior to
the purge on `/etc/httpd/conf.d` being evaluated so that the ssl.conf
file gets removed on the first client run.
@tphoney
Copy link
Contributor

tphoney commented Feb 22, 2016

Thanks for the change !

tphoney added a commit that referenced this pull request Feb 22, 2016
Ensure module packages are installed before evaluating conf.d dir
@tphoney tphoney merged commit ba2f386 into puppetlabs:master Feb 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants