Change ownership for byos config files

Signed-off-by: Luca Carrogu <[email protected]>

Author: lukeseawalker

cookbooks/aws-parallelcluster-scheduler-plugin/resources/execute_event_handler.rb

@@ -21,6 +21,7 @@
   event_log_err = node['cluster']['scheduler_plugin']['handler_log_err']
   event_cwd = node['cluster']['scheduler_plugin']['home']
   event_user = node['cluster']['scheduler_plugin']['user']
+  event_user_group = node['cluster']['scheduler_plugin']['group']
   event_timeout = 3600
   event_env = build_env
   event_log_prefix_error = "%Y-%m-%d %H:%M:%S,000 - [#{new_resource.event_name}] - ERROR:"
@@ -30,7 +31,7 @@
   # switch stderr/stdout with (2>&1 1>&3-), process error (now on stdout), switch back stdout/stderr with (3>&1 1>&2) and then process output
   event_command = Shellwords.escape("set -o pipefail; { (#{new_resource.event_command}) 2>&1 1>&3- | ts '#{event_log_prefix_error}' | tee -a #{event_log_err}; } " \
     "3>&1 1>&2 | ts '#{event_log_prefix_info}' | tee -a #{event_log_out}")
-  cmd = Mixlib::ShellOut.new("/bin/bash -c #{event_command}", user: event_user, group: event_user, login: true, env: event_env, cwd: event_cwd, timeout: event_timeout)
+  cmd = Mixlib::ShellOut.new("/bin/bash -c #{event_command}", user: event_user, group: event_user_group, login: true, env: event_env, cwd: event_cwd, timeout: event_timeout)
   cmd.run_command
 
   if cmd.error?
@@ -85,7 +86,9 @@ def build_env
       end
     end
 
-    FileUtils.cp(source_scheduler_plugin_substack_outputs, target_scheduler_plugin_substack_outputs) if ::File.exist?(source_scheduler_plugin_substack_outputs)
+    if ::File.exist?(source_scheduler_plugin_substack_outputs)
+      copy_config("scheduler plugin substack outputs", source_scheduler_plugin_substack_outputs, target_scheduler_plugin_substack_outputs)
+    end
 
     # Load static env from file or build it if file not found
     source_handler_env = "#{node['cluster']['shared_dir']}/handler-env.json"
@@ -109,7 +112,9 @@ def build_env
   def copy_config(config_type, source_config, target_config)
     raise "Expected #{config_type} file not found in (#{source_config})" unless ::File.exist?(source_config)
 
+    Chef::Log.info("Copying #{config_type} file from (#{source_config}) to (#{target_config})")
     FileUtils.cp(source_config, target_config)
+    FileUtils.chown(node['cluster']['scheduler_plugin']['user'], node['cluster']['scheduler_plugin']['group'], target_config)
   end
 
   def build_dynamic_env