Skip to content

Commit

Permalink
[REX SSH Plugin] Don't manage net-ssh-krb package
Browse files Browse the repository at this point in the history
The package is not required since smart_proxy_remote_execution_ssh-0.5.0
and is no longer packaged in foreman repositories
  • Loading branch information
wbclark committed Mar 23, 2022
1 parent bed270b commit 531bec4
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 10 deletions.
7 changes: 0 additions & 7 deletions manifests/plugin/remote_execution/ssh.pp
Original file line number Diff line number Diff line change
Expand Up @@ -53,13 +53,6 @@
listen_on => $listen_on,
}

if $ssh_kerberos_auth {
$kerberos_pkg = "${foreman_proxy::params::ruby_package_prefix}net-ssh-krb"
package { $kerberos_pkg:
ensure => present,
}
}

if $generate_keys {
class { 'foreman_proxy::plugin::remote_execution::ssh::keys':
install_key => $install_key,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
context "on #{os}" do
let(:facts) { os_facts }
let(:pre_condition) { 'include foreman_proxy' }
let(:package_name) { os.start_with?('debian') ? 'ruby-net-ssh-krb' : 'tfm-rubygem-net-ssh-krb' }

describe 'with default settings' do
it { should contain_class('foreman_proxy::plugin::dynflow') }
Expand All @@ -29,7 +28,6 @@
end

it { should_not contain_file('/root/.ssh') }
it { should_not contain_package(package_name) }
end

describe 'with override parameters' do
Expand Down Expand Up @@ -66,7 +64,6 @@

it { should_not contain_exec('generate_ssh_key') }
it { should_not contain_file('/root/.ssh') }
it { should contain_package(package_name) }
end

describe 'with ssh key generating and installation' do
Expand Down

0 comments on commit 531bec4

Please sign in to comment.