diff --git a/manifests/init.pp b/manifests/init.pp index 4da6b66..182fb02 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -147,14 +147,7 @@ class ssh ( $default_sshd_config_serverkeybits = '1024' } 'Solaris': { - $default_packages = ['SUNWsshcu', - 'SUNWsshdr', - 'SUNWsshdu', - 'SUNWsshr', - 'SUNWsshu'] $default_ssh_config_hash_known_hosts = undef - $default_ssh_package_source = '/var/spool/pkg' - $default_ssh_package_adminfile = undef $default_ssh_sendenv = false $default_ssh_config_forward_x11_trusted = undef $default_sshd_config_subsystem_sftp = '/usr/lib/ssh/sftp-server' @@ -167,14 +160,35 @@ class ssh ( $default_sshd_gssapicleanupcredentials = undef $default_sshd_acceptenv = false $default_sshd_config_serverkeybits = '768' + $default_ssh_package_adminfile = undef case $::kernelrelease { - '5.10','5.11': { - $default_service_name = 'ssh' - $default_service_hasstatus = true + '5.11': { + $default_packages = ['network/ssh', + 'network/ssh/ssh-key', + 'service/network/ssh'] + $default_service_name = 'ssh' + $default_service_hasstatus = true + $default_ssh_package_source = undef + } + '5.10': { + $default_packages = ['SUNWsshcu', + 'SUNWsshdr', + 'SUNWsshdu', + 'SUNWsshr', + 'SUNWsshu'] + $default_service_name = 'ssh' + $default_service_hasstatus = true + $default_ssh_package_source = '/var/spool/pkg' } '5.9' : { - $default_service_name = 'sshd' - $default_service_hasstatus = false + $default_packages = ['SUNWsshcu', + 'SUNWsshdr', + 'SUNWsshdu', + 'SUNWsshr', + 'SUNWsshu'] + $default_service_name = 'sshd' + $default_service_hasstatus = false + $default_ssh_package_source = '/var/spool/pkg' } default: { fail('ssh module supports Solaris kernel release 5.9, 5.10 and 5.11.') diff --git a/spec/classes/init_spec.rb b/spec/classes/init_spec.rb index 6f98d7a..0ee0765 100644 --- a/spec/classes/init_spec.rb +++ b/spec/classes/init_spec.rb @@ -139,12 +139,10 @@ describe 'ssh' do it { should_not contain_class('common')} - ['SUNWsshcu','SUNWsshdr','SUNWsshdu','SUNWsshr','SUNWsshu'].each do |pkg| + ['network/ssh','network/ssh/ssh-key','service/network/ssh'].each do |pkg| it { should contain_package(pkg).with({ 'ensure' => 'installed', - 'source' => '/var/spool/pkg', - 'adminfile' => nil, }) } end @@ -156,7 +154,7 @@ describe 'ssh' do 'owner' => 'root', 'group' => 'root', 'mode' => '0644', - 'require' => [ 'Package[SUNWsshcu]', 'Package[SUNWsshdr]', 'Package[SUNWsshdu]', 'Package[SUNWsshr]', 'Package[SUNWsshu]' ], + 'require' => [ 'Package[network/ssh]', 'Package[network/ssh/ssh-key]', 'Package[service/network/ssh]' ], }) } @@ -179,7 +177,7 @@ describe 'ssh' do 'owner' => 'root', 'group' => 'root', 'mode' => '0644', - 'require' => [ 'Package[SUNWsshcu]', 'Package[SUNWsshdr]', 'Package[SUNWsshdu]', 'Package[SUNWsshr]', 'Package[SUNWsshu]' ], + 'require' => [ 'Package[network/ssh]', 'Package[network/ssh/ssh-key]', 'Package[service/network/ssh]' ], }) } @@ -1580,7 +1578,7 @@ describe 'ssh' do let :facts do { :fqdn => 'monkey.example.com', :osfamily => 'Solaris', - :kernelrelease => '5.11', + :kernelrelease => '5.10', :sshrsakey => 'AAAAB3NzaC1yc2EAAAABIwAAAQEArGElx46pD6NNnlxVaTbp0ZJMgBKCmbTCT3RaeCk0ZUJtQ8wkcwTtqIXmmiuFsynUT0DFSd8UIodnBOPqitimmooAVAiAi30TtJVzADfPScMiUnBJKZajIBkEMkwUcqsfh630jyBvLPE/kyQcxbEeGtbu1DG3monkeymanOBW1AKc5o+cJLXcInLnbowMG7NXzujT3BRYn/9s5vtT1V9cuZJs4XLRXQ50NluxJI7sVfRPVvQI9EMbTS4AFBXUej3yfgaLSV+nPZC/lmJ2gR4t/tKvMFF9m16f8IcZKK7o0rK7v81G/tREbOT5YhcKLK+0wBfR6RsmHzwy4EddZloyLQ==' } end @@ -1601,7 +1599,7 @@ describe 'ssh' do let :facts do { :fqdn => 'monkey.example.com', :osfamily => 'Solaris', - :kernelrelease => '5.11', + :kernelrelease => '5.10', :sshrsakey => 'AAAAB3NzaC1yc2EAAAABIwAAAQEArGElx46pD6NNnlxVaTbp0ZJMgBKCmbTCT3RaeCk0ZUJtQ8wkcwTtqIXmmiuFsynUT0DFSd8UIodnBOPqitimmooAVAiAi30TtJVzADfPScMiUnBJKZajIBkEMkwUcqsfh630jyBvLPE/kyQcxbEeGtbu1DG3monkeymanOBW1AKc5o+cJLXcInLnbowMG7NXzujT3BRYn/9s5vtT1V9cuZJs4XLRXQ50NluxJI7sVfRPVvQI9EMbTS4AFBXUej3yfgaLSV+nPZC/lmJ2gR4t/tKvMFF9m16f8IcZKK7o0rK7v81G/tREbOT5YhcKLK+0wBfR6RsmHzwy4EddZloyLQ==' } end @@ -1660,7 +1658,7 @@ describe 'ssh' do let(:facts) do { :fqdn => 'monkey.example.com', :osfamily => 'Solaris', - :kernelrelease => '5.11', + :kernelrelease => '5.10', :sshrsakey => 'AAAAB3NzaC1yc2EAAAABIwAAAQEArGElx46pD6NNnlxVaTbp0ZJMgBKCmbTCT3RaeCk0ZUJtQ8wkcwTtqIXmmiuFsynUT0DFSd8UIodnBOPqitimmooAVAiAi30TtJVzADfPScMiUnBJKZajIBkEMkwUcqsfh630jyBvLPE/kyQcxbEeGtbu1DG3monkeymanOBW1AKc5o+cJLXcInLnbowMG7NXzujT3BRYn/9s5vtT1V9cuZJs4XLRXQ50NluxJI7sVfRPVvQI9EMbTS4AFBXUej3yfgaLSV+nPZC/lmJ2gR4t/tKvMFF9m16f8IcZKK7o0rK7v81G/tREbOT5YhcKLK+0wBfR6RsmHzwy4EddZloyLQ==' } end @@ -1681,7 +1679,7 @@ describe 'ssh' do let(:facts) do { :fqdn => 'monkey.example.com', :osfamily => 'Solaris', - :kernelrelease => '5.11', + :kernelrelease => '5.10', :sshrsakey => 'AAAAB3NzaC1yc2EAAAABIwAAAQEArGElx46pD6NNnlxVaTbp0ZJMgBKCmbTCT3RaeCk0ZUJtQ8wkcwTtqIXmmiuFsynUT0DFSd8UIodnBOPqitimmooAVAiAi30TtJVzADfPScMiUnBJKZajIBkEMkwUcqsfh630jyBvLPE/kyQcxbEeGtbu1DG3monkeymanOBW1AKc5o+cJLXcInLnbowMG7NXzujT3BRYn/9s5vtT1V9cuZJs4XLRXQ50NluxJI7sVfRPVvQI9EMbTS4AFBXUej3yfgaLSV+nPZC/lmJ2gR4t/tKvMFF9m16f8IcZKK7o0rK7v81G/tREbOT5YhcKLK+0wBfR6RsmHzwy4EddZloyLQ==' } end @@ -1696,7 +1694,7 @@ describe 'ssh' do let(:facts) do { :fqdn => 'monkey.example.com', :osfamily => 'Solaris', - :kernelrelease => '5.11', + :kernelrelease => '5.10', :sshrsakey => 'AAAAB3NzaC1yc2EAAAABIwAAAQEArGElx46pD6NNnlxVaTbp0ZJMgBKCmbTCT3RaeCk0ZUJtQ8wkcwTtqIXmmiuFsynUT0DFSd8UIodnBOPqitimmooAVAiAi30TtJVzADfPScMiUnBJKZajIBkEMkwUcqsfh630jyBvLPE/kyQcxbEeGtbu1DG3monkeymanOBW1AKc5o+cJLXcInLnbowMG7NXzujT3BRYn/9s5vtT1V9cuZJs4XLRXQ50NluxJI7sVfRPVvQI9EMbTS4AFBXUej3yfgaLSV+nPZC/lmJ2gR4t/tKvMFF9m16f8IcZKK7o0rK7v81G/tREbOT5YhcKLK+0wBfR6RsmHzwy4EddZloyLQ==' } end