Skip to content

Commit 9975fa0

Browse files
[test] Reenable install_munge_folders_created check
Among the original three folders checked, this commit removes checks on /var/run/munge. `/var/run` content changes across reboots. The check was failing 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 14d9023 commit 9975fa0

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)