Skip to content

Commit

Permalink
add support for plugin ensure stage *purged*
Browse files Browse the repository at this point in the history
the package resource type supports the following values for the
`ensure` parameter:

*    present
*    absent
*    purged
*    disabled
*    installed
*    latest
*    /./

this change adds support for *purged* as it has the same effect
in the context of this module.
  • Loading branch information
UiP9AV6Y committed Oct 14, 2024
1 parent cecabf1 commit 3152665
Show file tree
Hide file tree
Showing 4 changed files with 39 additions and 5 deletions.
4 changes: 3 additions & 1 deletion manifests/plugin.pp
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,10 @@
}

if $config {
$config_file_absent = $version in ['absent', 'purged']

file { $config_file:
ensure => bool2str($version == 'absent', 'absent', 'file'),
ensure => bool2str($config_file_absent, 'absent', 'file'),
owner => $config_file_owner,
group => $config_file_group,
mode => $config_file_mode,
Expand Down
10 changes: 6 additions & 4 deletions manifests/plugin/remote_execution/cockpit.pp
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,9 @@
Optional[String[1]] $ensure = undef,
Array[Stdlib::HTTPUrl] $origins = [],
) {
if $ensure != 'absent' {
$ensure_absent = $ensure in ['absent', 'purged']

unless $ensure_absent {
require foreman::plugin::remote_execution
}

Expand All @@ -36,7 +38,7 @@
version => $ensure,
}

if $ensure != 'absent' {
unless $ensure_absent {
service { 'foreman-cockpit':
ensure => running,
enable => true,
Expand All @@ -45,7 +47,7 @@
}
}

$file_ensure = bool2str($ensure == 'absent', 'absent', 'file')
$file_ensure = bool2str($ensure_absent, 'absent', 'file')

file { "${config_directory}/cockpit.conf":
ensure => $file_ensure,
Expand All @@ -65,7 +67,7 @@
require => Foreman::Plugin['remote_execution-cockpit'],
}

if $ensure == 'absent' {
if $ensure_absent {
foreman_config_entry { 'remote_execution_cockpit_url':
value => '',
ignore_missing => true,
Expand Down
16 changes: 16 additions & 0 deletions spec/classes/plugin/remote_execution_cockpit_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -139,6 +139,22 @@ class {'foreman':
it { is_expected.to contain_file('/etc/foreman/cockpit/foreman-cockpit-session.yml').with_ensure('absent') }
it { is_expected.to contain_foreman_config_entry('remote_execution_cockpit_url').with_value('') }
end

describe 'ensure purged' do
let(:params) do
{
ensure: 'purged',
}
end

it { is_expected.to compile.with_all_deps }
it { is_expected.not_to contain_class('foreman::plugin::remote_execution') }
it { is_expected.to contain_foreman__plugin('remote_execution-cockpit').with_version('purged') }
it { is_expected.not_to contain_service('foreman-cockpit') }
it { is_expected.to contain_file('/etc/foreman/cockpit/cockpit.conf').with_ensure('absent') }
it { is_expected.to contain_file('/etc/foreman/cockpit/foreman-cockpit-session.yml').with_ensure('absent') }
it { is_expected.to contain_foreman_config_entry('remote_execution_cockpit_url').with_value('') }
end
end
end
end
14 changes: 14 additions & 0 deletions spec/defines/foreman_plugin_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,20 @@
it { is_expected.to contain_package('myplugin').with_ensure('absent') }
it { is_expected.to contain_file('/etc/foreman/plugins/foreman_myplugin.yaml').with_ensure('absent') }
end

context 'ensure purged' do
let(:params) do
{
package: 'myplugin', # fixed to make testing easier
version: 'purged',
config: 'the config content',
}
end

it { is_expected.to compile.with_all_deps }
it { is_expected.to contain_package('myplugin').with_ensure('purged') }
it { is_expected.to contain_file('/etc/foreman/plugins/foreman_myplugin.yaml').with_ensure('absent') }
end
end
end
end

0 comments on commit 3152665

Please sign in to comment.