Merge pull request #215 from Arakmar/require_ssh_config

Make sure the ssh config dir is present before trying to define the known host file.
This commit is contained in:
Garrett Honeycutt 2017-05-19 11:18:21 -04:00 committed by GitHub
commit 7582138819
2 changed files with 17 additions and 5 deletions

View File

@ -932,6 +932,7 @@ class ssh (
owner => $ssh_config_global_known_hosts_owner,
group => $ssh_config_global_known_hosts_group,
mode => $ssh_config_global_known_hosts_mode,
require => Package[$packages_real],
}
# import all nodes' ssh keys

View File

@ -235,6 +235,12 @@ describe 'ssh' do
})
}
facts[:ssh_packages].each do |pkg|
it {
should contain_file('ssh_known_hosts').that_requires("Package[#{pkg}]")
}
end
it {
should contain_file('ssh_config').with({
'ensure' => 'file',
@ -1990,6 +1996,7 @@ describe 'ssh' do
'owner' => 'root',
'group' => 'root',
'mode' => '0644',
'require' => ['Package[openssh-server]', 'Package[openssh-clients]'],
})
}
@ -2086,6 +2093,7 @@ describe 'ssh' do
'owner' => 'gh',
'group' => 'root',
'mode' => '0644',
'require' => ['Package[openssh-server]', 'Package[openssh-clients]'],
})
}
end
@ -2112,6 +2120,7 @@ describe 'ssh' do
'owner' => 'root',
'group' => 'gh',
'mode' => '0644',
'require' => ['Package[openssh-server]', 'Package[openssh-clients]'],
})
}
end
@ -2138,6 +2147,7 @@ describe 'ssh' do
'owner' => 'root',
'group' => 'root',
'mode' => '0666',
'require' => ['Package[openssh-server]', 'Package[openssh-clients]'],
})
}
end
@ -2201,6 +2211,7 @@ describe 'ssh' do
'owner' => 'root',
'group' => 'root',
'mode' => '0644',
'require' => ['Package[openssh-server]', 'Package[openssh-clients]'],
})
}
end