Remove unused Munge SysV init logic

The Munge installation is already based on systemd. The SysV Init logic is no longer used. We forgot to remove this logic when we dropped support for Amazon Linux 1

When running kitchen tests on Docker, munge on some OSes still uses SysV. Therefore, we skip some `service` resource execution. We take the kitchen tests on EC2 instance as the golden standard. This is consistent with what we've been doing (i.e. We have been skip some `service` resource for other packages)

Signed-off-by: Hanwen <[email protected]>

Author: hanwen-cluster

cookbooks/aws-parallelcluster-slurm/libraries/helpers.rb

@@ -65,7 +65,7 @@ def enable_munge_service
     action %i(enable start)
     retries 5
     retry_delay 10
-  end
+  end unless on_docker?
 end
 
 def setup_munge_head_node

cookbooks/aws-parallelcluster-slurm/resources/munge/partial/_munge_actions.rb

@@ -48,7 +48,6 @@
     action_purge_packages
     action_download_source_code
     action_compile_and_install
-    action_update_init_script
     action_set_user_and_group
     action_create_required_directories
   }
@@ -93,19 +92,6 @@
   end
 end
 
-action :update_init_script do
-  # Updated munge init script for Amazon Linux
-  template '/etc/init.d/munge' do
-    source 'munge/munge-init.erb'
-    cookbook 'aws-parallelcluster-slurm'
-    owner 'root'
-    group 'root'
-    variables(munge_user: munge_user,
-              munge_group: munge_group)
-    mode '0755'
-  end
-end
-
 action :set_user_and_group do
   # Setup munge group
   group munge_group do

cookbooks/aws-parallelcluster-slurm/resources/munge_key_manager.rb

@@ -29,7 +29,7 @@ def restart_munge_service
     action :restart
     retries 5
     retry_delay 10
-  end
+  end unless on_docker?
 end
 
 def enable_munge_service
@@ -38,7 +38,7 @@ def enable_munge_service
     action :enable
     retries 5
     retry_delay 10
-  end
+  end unless on_docker?
 end
 
 def share_munge_key_to_dir(shared_dir)