Skip to content
This repository has been archived by the owner on Mar 5, 2024. It is now read-only.

Remove apache default vhost on debian #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jwennerberg
Copy link
Contributor

No description provided.

@ghoneycutt
Copy link
Contributor

Does this even work? I would think it would cause duplicate definitions with the apache module. Or does this file get created by the apache2 package on debian and is not managed by puppet?

I'm thinking we could also use something like

apache::vhost { '000-default':
  ensure => absent,
}

@ghoneycutt
Copy link
Contributor

Conditional logic indicates that we should also have a spec test..

@jwennerberg
Copy link
Contributor Author

Yes, this file is added by the apache2 package in Debian. Not puppet managed.

Using apache::vhost won't work since the 'default' file/symlink doesn't end with .conf. From vhost.pp:

  file { "${priority}-${name}.conf":
    ensure  => $ensure,

@ghoneycutt
Copy link
Contributor

What does that file do that's an issue?

We can merge the functionality with a note that it should be removed. Recommend submitting a patch to puppetlabs/apache that can remove the file. They will also want to know why it can be an issue, I would guess.

@jwennerberg
Copy link
Contributor Author

It's an issue because we want Observium to be the default site on port 80.

I think puppetlabs/apache already takes care of this by purging the vhost directory. The branch we're using doesn't do that. So I guess re-enabling that functionality is the way to go?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants