Skip to content

Commit

Permalink
Remove suppport for EL6
Browse files Browse the repository at this point in the history
According to metadata.json we no longer supported this so we can
simplify.
  • Loading branch information
ekohl committed Jan 6, 2017
1 parent 31926a8 commit fafad86
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 181 deletions.
2 changes: 1 addition & 1 deletion manifests/init.pp
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
# $db_password:: The Candlepin database password;
# default 'candlepin'
#
# $tomcat:: The system tomcat to use, tomcat6 on RHEL6 and tomcat on most Fedoras
# $tomcat:: The system tomcat to use
#
# $crl_file:: The Certificate Revocation File for Candlepin
#
Expand Down
11 changes: 1 addition & 10 deletions manifests/params.pp
Original file line number Diff line number Diff line change
@@ -1,15 +1,6 @@
# Candlepin params
class candlepin::params {
$tomcat = $::osfamily ? {
/^(RedHat|Linux)/ => $::operatingsystem ? {
'Fedora' => 'tomcat',
default => $::operatingsystemrelease ? {
/^7\./ => 'tomcat',
default => 'tomcat6'
}
},
default => 'tomcat'
}
$tomcat = 'tomcat'

$ssl_port = 8443

Expand Down
1 change: 0 additions & 1 deletion manifests/service.pp
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
enable => true,
hasstatus => true,
hasrestart => true,
require => [Service[$candlepin::db_type]],
}

if $candlepin::run_init == true {
Expand Down
3 changes: 1 addition & 2 deletions spec/classes/candlepin_config_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
on_supported_os.each do |os, facts|
context "on #{os}" do
let(:facts) { facts }
let(:tomcat_name) { os.include?('-6-') ? 'tomcat6' : 'tomcat' }

describe 'without parameters' do
let :pre_condition do
Expand Down Expand Up @@ -92,7 +91,7 @@
end

it "should setup the tomcat config file" do
should contain_file("/etc/#{tomcat_name}/server.xml").
should contain_file("/etc/tomcat/server.xml").
with_content(/^ <Connector port="9070" protocol="HTTP\/1.1" SSLEnabled="true"/).
with({})
end
Expand Down
4 changes: 2 additions & 2 deletions spec/classes/candlepin_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,13 @@
on_supported_os.each do |os, facts|
context "on #{os}" do
let(:facts) { facts }
let(:tomcat_name) { os.include?('-6-') ? 'tomcat6' : 'tomcat' }

it { should compile.with_all_deps }
it { should contain_class('candlepin::install') }
it { should contain_class('candlepin::config') }
it { should contain_class('candlepin::database') }
it { should contain_class('candlepin::service') }
it { should contain_service(tomcat_name).with_ensure('running') }
it { should contain_service('tomcat').with_ensure('running') }
end
end
end
165 changes: 0 additions & 165 deletions templates/tomcat6/server.xml.erb

This file was deleted.

0 comments on commit fafad86

Please sign in to comment.