openstack/puppet-ceph
Code Issues Proposed changes

Ensure public_addr option is purged

Browse Source 
... instead of leaving the option.

Change-Id: I7e4e2403570cff622d81804c13edf03548a0de18
This commit is contained in:
Takashi Kajinami 2023-07-23 02:32:03 +09:00
parent 8bc67e20e7
commit 1857b60b60
2 changed files with 10 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
manifests/mds.pp
View File

@ -109,11 +109,19 @@ class ceph::mds (
ceph_config {
"mds.${mds_id}/public_addr": value => $public_addr;
}
} else {
ceph_config {
"mds.${mds_id}/public_addr": ensure => absent;
}
}
} else {
ceph_config {
'mds/mds_data': ensure => absent;
'mds/keyring': ensure => absent;
}
ceph_config {
"mds.${mds_id}/public_addr": ensure => absent;
}
}
}

2
spec/classes/ceph_mds_spec.rb
View File

@ -55,12 +55,14 @@ describe 'ceph::mds' do
describe "not activated" do
let :params do
{
:mds_id => 'mymds',
:mds_activate => false
}
end
it { should contain_ceph_config('mds/mds_data').with_ensure('absent') }
it { should contain_ceph_config('mds/keyring').with_ensure('absent') }
it { should contain_ceph_config('mds.mymds/public_addr').with_ensure('absent') }
end
end