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
Use apache mod classes #516
Conversation
The pl-apache PR has been merged and rebased. Do we want to wait for a pl-apache release though? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it'll need to wait for the next apache release, then increase the metadata.json dep.
manifests/config/passenger.pp
Outdated
@@ -95,6 +95,13 @@ | |||
include ::apache::mod::headers | |||
include ::apache::mod::passenger | |||
|
|||
if $::foreman::ipa_authentication { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be added to the local class parameters list so it can be set in foreman::config::passenger tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated with tests
@@ -314,6 +319,41 @@ | |||
should contain_apache__vhost('foreman-ssl').with_passenger_pre_start("https://#{facts[:fqdn]}:8443") | |||
end | |||
end | |||
|
|||
describe 'with different ports set' do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with ipa_authentication => true
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's still early. Fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Subject to dep on puppetlabs-apache being increased on release.
Requires puppetlabs-apache 7301123aa51afe696f129988482889f61aa4f0d0 which is shipped since 2.0.0 so we bump the required version. [1] puppetlabs/puppetlabs-apache@7301123
Rebased now that puppetlabs-apache 2.0.0 is out. |
merged, bedankt @ekohl! |
Requires puppetlabs/puppetlabs-apache#1588