Skip to content

Commit cb0b0b8

Browse files
committed
Stop failing on unsupported platforms
This allows for the use of unsupported platforms by simply adding the correct hiera data. Examples can be found under `examples/hiera/`.
1 parent 809c0c8 commit cb0b0b8

File tree

2 files changed

+0
-52
lines changed

2 files changed

+0
-52
lines changed

manifests/init.pp

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -253,26 +253,6 @@
253253
Boolean $common_files_create_links = false,
254254
Optional[String] $common_files_suffix = undef,
255255
) {
256-
# Fail on unsupported platforms
257-
if $facts['os']['family'] == 'RedHat' and !($facts['os']['release']['major'] in ['2','5','6','7','8', '9']) {
258-
fail("osfamily RedHat's os.release.major is <${::facts['os']['release']['major']}> and must be 2, 5, 6, 7, 8 or 9")
259-
}
260-
261-
if $facts['os']['family'] == 'Solaris' and !($facts['kernelrelease'] in ['5.9','5.10','5.11']) {
262-
fail("osfamily Solaris' kernelrelease is <${facts['kernelrelease']}> and must be 5.9, 5.10 or 5.11")
263-
}
264-
265-
if $facts['os']['family'] == 'Suse' and !($facts['os']['release']['major'] in ['9','10','11','12','13','15']) {
266-
fail("osfamily Suse's os.release.major is <${::facts['os']['release']['major']}> and must be 9, 10, 11, 12, 13 or 15")
267-
}
268-
269-
if $facts['os']['name'] == 'Debian' and !($facts['os']['release']['major'] in ['7','8','9','10','11','12']) {
270-
fail("Debian's os.release.major is <${facts['os']['release']['major']}> and must be 7, 8, 9, 10, 11 or 12")
271-
}
272-
273-
if $facts['os']['name'] == 'Ubuntu' and !($facts['os']['release']['major'] in ['12.04', '14.04', '16.04', '18.04', '20.04', '22.04']) {
274-
fail("Ubuntu's os.release.major is <${facts['os']['release']['major']}> and must be 12.04, 14.04, 16.04, 18.04, 20.04 or 22.04")
275-
}
276256

277257
if $pam_d_sshd_template == 'pam/sshd.custom.erb' {
278258
unless $pam_sshd_auth_lines and

spec/classes/init_spec.rb

Lines changed: 0 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -352,36 +352,4 @@
352352
end
353353
end
354354
end
355-
356-
describe 'on unsupported platforms' do
357-
context 'with defaults params on Debian 6' do
358-
let(:facts) { { os: { 'name' => 'Debian', 'release' => { 'major' => '6' } } } }
359-
360-
it { is_expected.to compile.and_raise_error(%r{must be}) }
361-
end
362-
363-
context 'with defaults params on RedHat 4' do
364-
let(:facts) { { os: { 'family' => 'RedHat', 'release' => { 'major' => '4' } } } }
365-
366-
it { is_expected.to compile.and_raise_error(%r{must be}) }
367-
end
368-
369-
context 'with defaults params on Solaris 8' do
370-
let(:facts) { { os: { 'family' => 'Solaris' }, kernelrelease: '5.8' } }
371-
372-
it { is_expected.to compile.and_raise_error(%r{must be}) }
373-
end
374-
375-
context 'with defaults params on SLES 8' do
376-
let(:facts) { { os: { 'family' => 'Suse', 'release' => { 'major' => '8' } } } }
377-
378-
it { is_expected.to compile.and_raise_error(%r{must be}) }
379-
end
380-
381-
context 'with defaults params on Ubuntu 10.04' do
382-
let(:facts) { { os: { 'name' => 'Ubuntu', 'release' => { 'major' => '10.04' } } } }
383-
384-
it { is_expected.to compile.and_raise_error(%r{must be}) }
385-
end
386-
end
387355
end

0 commit comments

Comments
 (0)