From 49e20b99a155c27081b1b53ac7880cb5c89c36f0 Mon Sep 17 00:00:00 2001 From: Ryan Anderson Date: Fri, 4 Aug 2023 09:53:56 -0400 Subject: [PATCH] Move ulimit check back to config since it fails on jenkins --- .../test/controls/users_spec.rb | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/cookbooks/aws-parallelcluster-platform/test/controls/users_spec.rb b/cookbooks/aws-parallelcluster-platform/test/controls/users_spec.rb index a51e8a1775..47374b725d 100644 --- a/cookbooks/aws-parallelcluster-platform/test/controls/users_spec.rb +++ b/cookbooks/aws-parallelcluster-platform/test/controls/users_spec.rb @@ -18,9 +18,6 @@ describe limits_conf("/etc/security/limits.d/00_all_limits.conf") do its('*') { should include ['-', 'nofile', "10000"] } end - describe bash("sudo -u #{user} bash -c 'ulimit -Sn'") do - its('stdout') { should cmp >= '8192' } - end end control 'tag:config_admin_user_and_group_correctly_defined' do @@ -36,3 +33,11 @@ its('gid') { should eq node['cluster']['cluster_admin_group_id'] } end end + +control 'tag:config_ulimit_is_not_lower_than_8192' do + only_if { !instance.custom_ami? } + + describe bash("sudo -u #{user} bash -c 'ulimit -Sn'") do + its('stdout') { should cmp >= '8192' } + end +end