beaker-pe 1.12.0 → 1.12.1

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    YTI4NjI0OGUyOGY5ZGMyZDljNTBhMjA1NDhhMzYxY2I5M2JjYmVkNQ==
+    ZmM1MjMxNjZkYjU3ZWM2ZGI4NDhjMmI0ZWYwM2FjOGZkZmQ0NmNmMA==
   data.tar.gz: !binary |-
-    YWIyYThhODVhMmU0OGQ4ZjA2ZmM1NmQ3MzBhYjdiMTViMzhjZjgzMQ==
+    YmI3YmJlOWE2NjYwYzUwNTBhYTMyYWQyYTY5OGVmMTFmN2YwMmNmNA==
 SHA512:
   metadata.gz: !binary |-
-    MjkzODE5YmVlMzU1YjUzMTVkYzhiMTlhNjgyMTQ4MDkxOGMxMDcxMmU2ZjJj
-    NmI0MWE2MGJmNGFkNDExMDFlMzgzMmRiMDEyODJmNzI0MmZkMjc5Zjc2MmUz
-    OWIxYjIzYzRjMzFkMDJhZTE0MmFmZDFjZGFhMTFmZDU2Y2MxNjI=
+    NTVjYTNmOTJkOWY0YzVmNjUyOGM0MjAyMzVlZmVhZDI5ZTk3NjhlMjJjOTNm
+    NmI0YzZlYTMwYWIxYTM1M2FkZTJlYzIwYzlhZTY2M2Q4YTYwNDM1MTRmNTdl
+    NjQwY2YxZGY0Y2FkMzZmZTk0OWU1MDUzZDJiZTQ3YWM4NTU4ODA=
   data.tar.gz: !binary |-
-    YzdjYTQyOTI5ZDBmYmVjMzZiYjRmNmJmZmE1NmRmZTY5ZDY4OTA3ZGMyMzcx
-    YTAyMWM1ZmVjNTIzMzk2MDdmYzhmZTc1OWVmNDg3MWJjMjBjYWY2NDUwZDM1
-    NzZlYWRjMGJhNGJiMmMwNjU2YzJiY2JmN2E5NzVjMjZiNGQ2YzY=
+    NzBhYTVjOTRmMjc5ODRhZWExNjliY2VkMmI5ODk3YzExODQ3YmIxMWMwMGYw
+    NDBjOTQwZjY5ZTQ1ZGE2MTk5MDJjNWQ0MDNhNWY2ODI5YTgzZGM3ZGY3MGU4
+    ODQxNGFjNzYzODlhMjMxYWVjMDU3NjAyZTMyMGM3ODAzYjg5ODA=

data/HISTORY.md

@@ -1,6 +1,7 @@
 # worker - History
 ## Tags
-* [LATEST - 23 Mar, 2017 (dd68cfa4)](#LATEST)
+* [LATEST - 29 Mar, 2017 (95b0e94f)](#LATEST)
+* [1.12.0 - 23 Mar, 2017 (0784adc6)](#1.12.0)
 * [1.11.0 - 23 Mar, 2017 (6c3b0067)](#1.11.0)
 * [1.10.0 - 20 Mar, 2017 (22e22ca8)](#1.10.0)
 * [1.9.1 - 22 Feb, 2017 (3b0bd457)](#1.9.1)
@@ -34,7 +35,34 @@
 * [0.1.0 - 29 Feb, 2016 (4fc88d8c)](#0.1.0)
 
 ## Details
-### <a name = "LATEST">LATEST - 23 Mar, 2017 (dd68cfa4)
+### <a name = "LATEST">LATEST - 29 Mar, 2017 (95b0e94f)
+
+* (GEM) update beaker-pe version to 1.12.1 (95b0e94f)
+
+* Merge pull request #65 from nicklewis/BKR-1085-sign-certs-before-stopping-agent (e1b6d137)
+
+
+```
+Merge pull request #65 from nicklewis/BKR-1085-sign-certs-before-stopping-agent
+
+(BKR-1085) Sign certs *before* stopping agent service
+```
+* (BKR-1085) Sign certs *before* stopping agent service (3b5208eb)
+
+
+```
+(BKR-1085) Sign certs *before* stopping agent service
+
+Previously, the simple_monolithic install method was installing agents,
+stopping the agent service and then attempting to sign certificates.
+This was failing sporadically because the agent service needs to run
+enough to submit a certificate request before being stopped. This was
+being handled properly in the generic install method, but the order of
+operations was swapped in the monolithic install method.
+```
+### <a name = "1.12.0">1.12.0 - 23 Mar, 2017 (0784adc6)
+
+* (HISTORY) update beaker-pe history for gem release 1.12.0 (0784adc6)
 
 * (GEM) update beaker-pe version to 1.12.0 (dd68cfa4)
 

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

@@ -471,15 +471,15 @@ module Beaker
             end
           end
 
-          step "Stop puppet agents to avoid interfering with tests" do
-            stop_agent_on(all_hosts, :run_in_parallel => true)
-          end
-
           step "Sign agent certificates" do
             # This will sign all cert requests
             sign_certificate_for(agents)
           end
 
+          step "Stop puppet agents to avoid interfering with tests" do
+            stop_agent_on(all_hosts, :run_in_parallel => true)
+          end
+
           step "Run puppet to setup mcollective and pxp-agent" do
             on all_hosts, puppet_agent('-t'), :acceptable_exit_codes => [0,2], :run_in_parallel => true
 

data/lib/beaker-pe/version.rb

@@ -3,7 +3,7 @@ module Beaker
     module PE
 
       module Version
-        STRING = '1.12.0'
+        STRING = '1.12.1'
       end
 
     end

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

@@ -1293,16 +1293,10 @@ describe ClassMixedWithDSLInstallUtils do
       subject.simple_monolithic_install(monolithic, [el_agent, el_agent, deb_agent, deb_agent])
     end
 
-    it 'signs all certificates at once' do
+    it 'signs certificates then stops agents to avoid interference with tests' do
       agents = [el_agent, el_agent, deb_agent, deb_agent]
-      expect(subject).to receive(:sign_certificate_for).with(agents)
-
-      subject.simple_monolithic_install(monolithic, agents)
-    end
-
-    it 'stops the agents in parallel to avoid interference with tests' do
-      agents = [el_agent, el_agent, deb_agent, deb_agent]
-      expect(subject).to receive(:stop_agent_on).with([monolithic, *agents], :run_in_parallel => true)
+      expect(subject).to receive(:sign_certificate_for).with(agents).ordered
+      expect(subject).to receive(:stop_agent_on).with([monolithic, *agents], :run_in_parallel => true).ordered
 
       subject.simple_monolithic_install(monolithic, agents)
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: beaker-pe
 version: !ruby/object:Gem::Version
-  version: 1.12.0
+  version: 1.12.1
 platform: ruby
 authors:
 - Puppetlabs
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-03-23 00:00:00.000000000 Z
+date: 2017-03-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec