Add vhost "enabled" option to enable or disable it

[BDelacour]

Hello,

I need to manually handle if a vhost is enabled and I don't want Puppet to overwrite it. Coupled with apache::purge_vhost_dir: false, it works great !

Thank you !

[puppet-community-rangefinder[bot]]

apache::vhost is a type

Breaking changes to this file WILL impact these 129 modules (exact match):

• nwolfe-loadtest
• kritz-vagrantlamp
• garethr-wackopicko
• crayfishx-puppetdb_rundeck
• Firebladee-moodle
• puppetlabs-dashboard
• lightoze-bacula
• thomasvandoren-pypi
• glarizza-profiles
• binford2k-drupal
• fsalum-dashboard
• openstackci-dashboard
• xdrum-puppet
• cnacorrea-puppet
• boyand-graphite
• thejandroman-grafana
• natewalck-munki_appliance
• danieldreier-puppet_installer
• puppetlabs-awsdemo_profiles
• Aethylred-apaxy
• martialblog-limesurvey
• landcareresearch-pidservice
• premkumarp-myapp
• pkumarnath-myapp
• puppetlabs-wordpress_app
• kumarmks6-myapp
• snaragoni-myapp
• sasireddyfm-myapp
• tejoyasha-myapp
• gardouille-xymon
• spotify-talos
• norisnetwork-ceph
• vinish098-myapp
• pwaghray-myapp
• ranjithsimple-myapp
• ganeshselva95-myapp
• syellapu-myapp
• jgazeley-speedtest
• garystafford-apache_example_config
• arden-local_rpm_repo
• glarizza-osx_management
• hgkamath-owncloud
• brucem-ezpublish
• gajdaw-django_app
• garethr-graphite
• thoherr-railsapp
• southernhill-phpldapadmin
• rjbalest-praxis
• gene1wood-ipquery
• gajdaw-phpmyadmin
• 42ways-railsapp
• ploperations-puppet
• saw-reviewboard
• wyrie-nagiosql
• gerardkok-reposado
• dmcnicks-sympa
• leonardothibes-phpmyadmin
• Aethylred-puppetdashboard
• pinguinag-reprepro
• cirrax-autoconfigmail
• aimonb-nexusis_gerrit
• rendhalver-icinga
• stephenrjohnson-puppet
• ploperations-github
• katello-crane
• covermymeds-sentry
• devika-myapp
• zordrak-puppet
• ghoneycutt-yum
• johanek-redmine
• abstractit-puppet
• openstack-ceph
• zivtech-drupal_php
• cristifalcas-kibana
• puppet-smokeping
• objectiflibre-puppet
• SchnWalter-happydev
• openstack-openstacklib
• openstack-ironic
• katello-foreman_proxy_content
• abstractit-icinga
• camlow325-grafanadash
• thejandroman-kibana3
• eshamow-simpleid
• fheinle-webserver
• mricon-bugzilla
• monkygames-beansbooks
• eschiller-trac
• thexa4-secrets_server
• bp85-mirror_repos
• mtsinc1-trac
• monkygames-landscape
• martasd-mediawiki
• tomkrieger-pxe_install
• jgazeley-nagios
• hexmode-mediawiki
• sgnl05-racktables
• benjaminrobertson-observium
• leonardothibes-usvn
• marcdeop-ratticdb
• cirrax-roundcube
• urgi-galaxy_roles_profiles
• puppetlabs-mrepo
• seteam-role
• jlondon-phpmyadmin
• datacentred-nfsen
• pltraining-classroom_legacy
• aimonb-nexusis_mediawiki
• gajdaw-symfony
• katello-capsule
• puppet-mrepo
• stackforge-keystone
• theforeman-pulpcore
• puppet-zabbix
• theforeman-foreman
• landcareresearch-ckan
• wdijkerman-zabbix
• abiquo-abiquo
• nnutter-testdb
• fancyolaya-dokuwiki
• zyronix-dokuwiki
• shoekstra-owncloud
• hetzner-roundcube
• geoffwilliams-r_profile
• Aethylred-gitlab
• katello-pulp
• stesie-gluon
• icann-webapp
• cirrax-postfixadmin

Breaking changes to this file MAY impact these 34 modules (near match):

• Azcender-balancer
• grahamgilbert-crypt
• jbussdieker-graphite_web
• nwaller-mailman
• hunner-gitolite
• tedivm-hieratic
• covermymeds-ruby
• spotify-puppetexplorer
• alexggolovin-opencart
• Lavaburn-cabot
• opuscodium-taiga
• camptocamp-catalog_diff
• nibalizer-puppetboard
• example42-kibana
• puppet-php
• puppet-puppetboard
• echoes-wrappers
• soli-wrappers
• openstack-tripleo
• example42-graylog2
• example42-tomcat
• jcalles-php
• mayflower-php
• gajdaw-php5
• samuelson-dockeragent
• pieterdp-hx_website
• puppetfinland-librenms
• pltraining-dockeragent
• rspiak-racktables
• zyronix-ipsets
• evenup-graphite
• example42-wordpress
• example42-foreman
• example42-puppet

This module is declared in 173 of 578 indexed public Puppetfiles.

---

These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.

Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.

[CLAassistant]

All committers have signed the CLA.

[github-actions[bot]]

This PR has been marked as stale because it has been open for a while and has had no recent activity. If this PR is still important to you please drop a comment below and we will add this to our backlog to complete. Otherwise, it will be closed in 7 days.

[david22swan]

@BDelacour Hey, sorry it's taken so long to comment on this Anyway the change look's ok to me, but would prefer it to have a more descriptive name, i.e. vhost_enabled, and some documentation added to it at the top alongside what already exists for the other variables, so that it is represented in the documentation. As an additional note we would also require it to be given a type.

[BDelacour]

Hey ! Sorry I left this PR stale for some time. I'll update it soon

[github-actions[bot]]

Hello! 👋

This pull request has been open for a while and has had no recent activity. We've labelled it with attention-needed so that we can get a clear view of which PRs need our attention.

If you are waiting on a response from us we will try and address your comments on a future Community Day.

Alternatively, if it is no longer relevant to you please close the PR with a comment.

Please note that if a pull request receives no update for 7 after it has been labelled, it will be closed. We are always happy to re-open pull request if they have been closed in error.