Skip to content

Commit db54f1b

Browse files
author
Himani Anil Deshpande
committed
Make cfn-hup-update-action.sh executable Only by root
1 parent 1fc4461 commit db54f1b

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

cookbooks/aws-parallelcluster-environment/resources/cfn_hup_configuration.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@
8787
source "cfn_hup_configuration/#{node['cluster']['node_type']}/cfn-hup-update-action.sh.erb"
8888
owner 'root'
8989
group 'root'
90-
mode '0744' # TODO: Change permission
90+
mode '0700'
9191
variables(
9292
monitor_shared_dir: monitor_shared_dir,
9393
launch_template_resource_id: node['cluster']['launch_template_id']

cookbooks/aws-parallelcluster-environment/spec/unit/resources/cfn_hup_configuration_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ def self.configure(chef_run)
9090
.with(source: "cfn_hup_configuration/#{node_type}/cfn-hup-update-action.sh.erb")
9191
.with(user: "root")
9292
.with(group: "root")
93-
.with(mode: "0744")
93+
.with(mode: "0700")
9494
.with(variables: {
9595
monitor_shared_dir: "#{MONITOR_SHARED_DIR}/dna",
9696
launch_template_resource_id: LAUNCH_TEMPLATE_ID,

cookbooks/aws-parallelcluster-environment/test/controls/cfn_hup_configuration_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@
5353

5454
describe file("#{node['cluster']['scripts_dir']}/cfn-hup-update-action.sh") do
5555
it { should exist }
56-
its('mode') { should cmp '0744' }
56+
its('mode') { should cmp '0700' }
5757
its('owner') { should eq 'root' }
5858
its('group') { should eq 'root' }
5959
end

0 commit comments

Comments
 (0)