[release 3.9] Reverting the PATH for cfnbootstrap (#2738)

* [Bug] Adding different values for `kernel-open` option * Revert "Changing the PATH order we set for using the cfnbootstrap script we have installed in virtualenv rather than using default scripts (#2730)" This reverts commit 26a3c9d. --------- Co-authored-by: Himani Deshpande <[email protected]> Co-authored-by: Enrico Usai <[email protected]>
aws · May 27, 2024 · 8eb077b · 8eb077b
1 parent 0a5759e
commit 8eb077b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/cookbooks/aws-parallelcluster-environment/templates/cfn_bootstrap/pcluster.sh.erb b/cookbooks/aws-parallelcluster-environment/templates/cfn_bootstrap/pcluster.sh.erb
@@ -2,6 +2,7 @@
 # pcluster.sh:
 #   Setup ParallelCluster environment variables
 #
-PATH=<%= @cfn_bootstrap_virtualenv_path %>/bin:$PATH
+
+PATH=$PATH:<%= @cfn_bootstrap_virtualenv_path %>/bin
 
 export PATH
diff --git a/cookbooks/aws-parallelcluster-environment/test/controls/cfn_bootstrap_spec.rb b/cookbooks/aws-parallelcluster-environment/test/controls/cfn_bootstrap_spec.rb
@@ -36,12 +36,6 @@
     its('mode') { should cmp '0644' }
     its('owner') { should eq 'root' }
     its('group') { should eq 'root' }
-    its('content') { should match "PATH=#{pyenv_dir}/versions/#{cfn_python_version}/envs/cfn_bootstrap_virtualenv/bin:\\$PATH" }
+    its('content') { should match "PATH=\\$PATH:#{pyenv_dir}/versions/#{cfn_python_version}/envs/cfn_bootstrap_virtualenv/bin" }
   end
-
-  desc "cfn-init needs to be from the cfnbootstrap virtualenv"
-  describe bash("sudo -u #{node['cluster']['cluster_user']} -i which cfn-init") do
-    its('exit_status') { should eq(0) }
-    its('stdout') { should match("#{pyenv_dir}/versions/#{cfn_python_version}/envs/cfn_bootstrap_virtualenv/bin/cfn-init") }
-  end unless os_properties.on_docker?
 end