include apache::mod::setenvif

[wbclark]

puppet-katello included this as of
theforeman/puppet-katello@afc08e3

[wbclark]

This is needed due to theforeman/foreman-installer#754 and fixed in katello with theforeman/puppet-katello#445

[sjha4]

Not to hijack this PR but putting it here since we have infrastructure gurus on the thread. There's another issue in the repo that is breaking dev boxes:

The package 'cyrus-sasl-plain' is declared twice for dev box:

1. https://github.com/theforeman/puppet-katello_devel/blob/master/manifests/install.pp
2. https://github.com/theforeman/puppet-qpid/blob/master/manifests/install.pp#L21

This throws a duplicate declaration error on vagrant up of dev boxes.

[chris1984]

Not to hijack this PR but putting it here since we have infrastructure gurus on the thread. There's another issue in the repo that is breaking dev boxes:

The package 'cyrus-sasl-plain' is declared twice for dev box:

1. https://github.com/theforeman/puppet-katello_devel/blob/master/manifests/install.pp
2. https://github.com/theforeman/puppet-qpid/blob/master/manifests/install.pp#L21

This throws a duplicate declaration error on vagrant up of dev boxes.

Can confirm I hit this as well, I usually have to have vagrant not delete the dev box and i go in and delete it out of qpid then the installer runs fine.

[sjha4]

Is the test failure related? We should merge this if not.