diff --git a/manifests/install/cni_plugins.pp b/manifests/install/cni_plugins.pp index ea3bbc7..218611f 100644 --- a/manifests/install/cni_plugins.pp +++ b/manifests/install/cni_plugins.pp @@ -24,29 +24,16 @@ case $method { 'tarball', 'loose': { - file { '/opt/cni-plugins': - ensure => stdlib::ensure($ensure, 'directory'), - purge => true, - recurse => true, - force => true, - } - $_url = k8s::format_url($download_url_template, { version => $version, }) - $_target = "/opt/cni-plugins/${version}"; - $_tarball_target = '/opt/cni-plugins/archives'; + $_target = "/opt/k8s/cni-${version}"; + $_tarball_target = '/opt/k8s/archives'; file { $_target: ensure => stdlib::ensure($ensure, 'directory'), } - file { $_tarball_target: - ensure => stdlib::ensure($ensure, 'directory'), - purge => true, - recurse => true, - } - archive { 'cni-plugins': ensure => $ensure, path => "${_tarball_target}/cni-plugins-linux-${version}.tgz", diff --git a/manifests/install/crictl.pp b/manifests/install/crictl.pp index 7b1b644..6b23444 100644 --- a/manifests/install/crictl.pp +++ b/manifests/install/crictl.pp @@ -31,27 +31,16 @@ $config_require = Package[$pkg] } else { - file { '/opt/cri-tools': - ensure => stdlib::ensure($ensure, 'directory'), - purge => true, - recurse => true, - force => true, - } - $_url = k8s::format_url($download_url_template, { version => $version, }) - $_target = "/opt/cri-tools/${version}"; - $_tarball_target = '/opt/cri-tools/archives'; + $_target = "/opt/k8s/crictl-${version}"; + $_tarball_target = '/opt/k8s/archives'; file { $_target: ensure => stdlib::ensure($ensure, 'directory'), } - file { $_tarball_target: - ensure => stdlib::ensure($ensure, 'directory'), - } - archive { 'crictl': ensure => $ensure, path => "${_tarball_target}/crictl-${version}-linux.tar.gz", diff --git a/manifests/server/etcd/setup.pp b/manifests/server/etcd/setup.pp index f1159e2..b54a459 100644 --- a/manifests/server/etcd/setup.pp +++ b/manifests/server/etcd/setup.pp @@ -78,26 +78,15 @@ Optional[Integer[0, 65535]] $gid = undef, ) { if $install == 'archive' { - file { '/opt/etcd': - ensure => stdlib::ensure($ensure, 'directory'), - purge => true, - recurse => true, - force => true, - } - $_url = k8s::format_url($archive_template, { version => $version, }) $_file = basename($_url) - $_target = "/opt/etcd/${version}"; - $_tarball_target = '/opt/etcd/archives'; + $_target = "/opt/k8s/etcd-${version}"; + $_tarball_target = '/opt/k8s/archives'; file { $_target: ensure => stdlib::ensure($ensure, 'directory'), } - file { $_tarball_target: - ensure => stdlib::ensure($ensure, 'directory'), - } - archive { 'etcd': ensure => $ensure, path => "${_tarball_target}/${_file}", diff --git a/spec/classes/server/etcd/setup_spec.rb b/spec/classes/server/etcd/setup_spec.rb index 024d6a5..920b30e 100644 --- a/spec/classes/server/etcd/setup_spec.rb +++ b/spec/classes/server/etcd/setup_spec.rb @@ -33,27 +33,27 @@ class { '::k8s::server::etcd': it do is_expected.to contain_archive('etcd').with( ensure: 'present', - path: '/opt/etcd/archives/etcd-v3.6.0-linux-amd64.tar.gz', + path: '/opt/k8s/archives/etcd-v3.6.0-linux-amd64.tar.gz', source: 'https://storage.googleapis.com/etcd/v3.6.0/etcd-v3.6.0-linux-amd64.tar.gz', extract: true, extract_command: 'tar xfz %s --strip-components=1', - extract_path: '/opt/etcd/3.6.0', + extract_path: '/opt/k8s/etcd-3.6.0', cleanup: true, - creates: ['/opt/etcd/3.6.0/etcd', '/opt/etcd/3.6.0/etcdctl'] + creates: ['/opt/k8s/etcd-3.6.0/etcd', '/opt/k8s/etcd-3.6.0/etcdctl'] ) is_expected.to contain_file('/usr/local/bin/etcd').with( ensure: 'link', mode: '0755', replace: true, - target: '/opt/etcd/3.6.0/etcd' + target: '/opt/k8s/etcd-3.6.0/etcd' ).that_notifies('Service[etcd]') is_expected.to contain_file('/usr/local/bin/etcdctl').with( ensure: 'link', mode: '0755', replace: true, - target: '/opt/etcd/3.6.0/etcdctl' + target: '/opt/k8s/etcd-3.6.0/etcdctl' ) end