Skip to content

Commit fba07e3

Browse files
[test] Reenable install_munge_folders_created check
Among the original three folders checked, this commit removes the check on `/var/run/munge`. `/var/run` content changes across reboots. The check was successful on validate phase because there was no reboot between validate phase and build phase. The check fails on test phase because build and test are running on different instances Signed-off-by: Hanwen <[email protected]>
1 parent 7d38d20 commit fba07e3

File tree

1 file changed

+13
-19
lines changed

1 file changed

+13
-19
lines changed

cookbooks/aws-parallelcluster-slurm/test/controls/munge_spec.rb

Lines changed: 13 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -35,25 +35,19 @@
3535
end
3636
end unless os_properties.redhat_on_docker?
3737

38-
# FIXME: Re-enabled the following check and fix failures
39-
# control 'tag:install_munge_folders_created' do
40-
# title 'Munge folder have been created'
41-
#
42-
# describe file('/var/log/munge') do
43-
# it { should exist }
44-
# it { should be_directory }
45-
# end
46-
#
47-
# describe file('/etc/munge') do
48-
# it { should exist }
49-
# it { should be_directory }
50-
# end
51-
#
52-
# describe file('/var/run/munge') do
53-
# it { should exist }
54-
# it { should be_directory }
55-
# end
56-
# end unless os_properties.redhat_on_docker?
38+
control 'tag:install_munge_folders_created' do
39+
title 'Munge folder have been created'
40+
41+
describe file('/var/log/munge') do
42+
it { should exist }
43+
it { should be_directory }
44+
end
45+
46+
describe file('/etc/munge') do
47+
it { should exist }
48+
it { should be_directory }
49+
end
50+
end unless os_properties.redhat_on_docker?
5751

5852
control 'tag:config_munge_service_enabled' do
5953
only_if { node['cluster']['scheduler'] == 'slurm' && !os_properties.redhat_on_docker? }

0 commit comments

Comments
 (0)