beaker-pe 2.11.21 → 2.11.22

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 527fe6d6360dfb51c8f1e2dbd9f032802b8b41b27f0c46e4ac80e96df585014e
-  data.tar.gz: 7c6a69a2b023551ca1332b2319583fba658536194d228fa29478bf33f906b58c
+  metadata.gz: 11cedfa7e51e26dc8c86bba1f90f58246281a4ecd807bf478257faf65ab6222a
+  data.tar.gz: 11aa7c6788e083087b365f3ece9c776152192729620d7c445991f28321daf10f
 SHA512:
-  metadata.gz: 1626c5e8d6cabd70eadef2c69262f1c3af5364092abe76f395198726c6aa98c86a273308edb2dbd400b3e77647a75f567bb7551d1c7f7e20cf4ed9e1cee958ed
-  data.tar.gz: a9f9764d8fdeaf123d498f90c63b571d7f759ce3b776022640a24dd6e91eefe84d18232ee992ea7cc074694a9c984791e4683f0497b453d34d5d9842fe989611
+  metadata.gz: f4b5e2f23e69e501026ac935cbf9d1f921f9bcc46c1547a192f89a7dcb83b867f3e24390ce270740b82fb40a8709340b74ec9ac2a1fefa7590b287b25f540df8
+  data.tar.gz: 9d98e876622f630b5d300c12e4a19cf8dbe08b4ba0a44cb5ec45f6bcc5c21d9c6af6d24bbd024b5c0da9688182364f7528fc9cf185efce9d8c96a9a68d8044d7

data/lib/beaker-pe/install/pe_utils.rb

@@ -144,8 +144,17 @@ module Beaker
         end
 
         #Return '--waitforlock 1' if the agent is >= 6.16 which was used in 2019.8.0.
-        def waitforlock_flag
-          version_is_less(master['pe_ver'], '2019.8.0') ? '' : '--waitforlock 1'
+        #In the test running with legacy agents, the pe_ver value on that host
+        #corresponds to the PE version the old agent comes from. Also, puppet_agent_version
+        #is present on these host objects.
+        def waitforlock_flag(host)
+          agent_version = host['puppet_agent_version']
+          pe_ver = host['pe_ver']
+          if (agent_version && version_is_less(agent_version, '6.16.0')) || version_is_less(host['pe_ver'], '2019.8.0')
+            ''
+          else
+            '--waitforlock 1'
+          end
         end
 
         # Generate the command line string needed to from a frictionless puppet-agent
@@ -731,7 +740,7 @@ module Beaker
           # waitforlock in case stopping the agent run after stopping the agent service
           # takes a little longer than usual
           step "Run puppet to setup mcollective and pxp-agent" do
-            on(master, puppet_agent("-t #{waitforlock_flag}"), :acceptable_exit_codes => [0,2])
+            on(master, puppet_agent("-t #{waitforlock_flag(master)}"), :acceptable_exit_codes => [0,2])
           end
 
           install_agents_only_on(agents, opts)
@@ -982,7 +991,7 @@ module Beaker
               install_hosts.each do |host|
                 # waitforlock in case stopping the agent run after stopping the agent service
                 # takes a little longer than usual
-                on host, puppet_agent("-t #{waitforlock_flag}"), :acceptable_exit_codes => [0,2]
+                on host, puppet_agent("-t #{waitforlock_flag(host)}"), :acceptable_exit_codes => [0,2]
 
                 # Workaround for PE-1105 when deploying 3.0.0
                 # The installer did not respect our database host answers in 3.0.0,
@@ -1023,7 +1032,7 @@ module Beaker
               install_hosts.each do |host|
                 # waitforlock in case stopping the agent run after stopping the agent service
                 # takes a little longer than usual
-                on host, puppet_agent("-t #{waitforlock_flag}"), :acceptable_exit_codes => [0,2]
+                on host, puppet_agent("-t #{waitforlock_flag(host)}"), :acceptable_exit_codes => [0,2]
                 # To work around PE-14318 if we just ran puppet agent on the
                 # database node we will need to wait until puppetdb is up and
                 # running before continuing
@@ -2128,7 +2137,9 @@ EOM
              # waitforlock in case stopping the agent run after stopping the agent service
              # takes a little longer than usual
              step "Run puppet on all agent nodes" do
-               on agent_nodes, puppet_agent("-t #{waitforlock_flag}"), :acceptable_exit_codes => [0,2], :run_in_parallel => true
+               block_on(agent_nodes, {:run_in_parallel => true}) do |host|
+                 on host, puppet_agent("-t #{waitforlock_flag(host)}"), :acceptable_exit_codes => [0,2]
+               end
              end
 
              #Workaround for windows frictionless install, see BKR-943

data/lib/beaker-pe/version.rb

@@ -3,7 +3,7 @@ module Beaker
     module PE
 
       module Version
-        STRING = '2.11.21'
+        STRING = '2.11.22'
       end
 
     end

data/spec/beaker-pe/install/pe_utils_spec.rb

@@ -2026,7 +2026,7 @@ describe ClassMixedWithDSLInstallUtils do
 
     it 'installs agent on agent hosts' do
       agents = [agent, agent]
-      expect(subject).to receive(:block_on).with(agents, :run_in_parallel => true)
+      expect(subject).to receive(:block_on).with(agents, :run_in_parallel => true).twice
       subject.install_agents_only_on(agents, opts)
     end
 
@@ -2036,13 +2036,6 @@ describe ClassMixedWithDSLInstallUtils do
       expect(subject).to receive(:stop_agent_on).with(agents, :run_in_parallel => true)
       subject.install_agents_only_on(agents, opts)
     end
-
-    it 'runs puppet on agent hosts' do
-      agents = [agent, agent]
-      expect(subject).to receive(:on).with(agents, proc {
-        |cmd| cmd.command == "puppet agent"}, hash_including(:run_in_parallel => true)).once
-      subject.install_agents_only_on(agents, opts)
-    end
   end
 
   describe 'do_higgs_install' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: beaker-pe
 version: !ruby/object:Gem::Version
-  version: 2.11.21
+  version: 2.11.22
 platform: ruby
 authors:
 - Puppetlabs
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-09-26 00:00:00.000000000 Z
+date: 2022-09-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec