beaker-pe 3.6.2 → 3.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 82fe681f6a5d8f304089ca2dca03e4e9080cbf502b0665e072103dac9e9e4aa9
-  data.tar.gz: 22f5065db00df1654fdc261de80272b0a3719db9fc6e2027b0f92d3e300f90e8
+  metadata.gz: 03bf3c005114d8ea4bc9e8a2f10b1d25afcd1e3919d09a0a079e925ef35b51ff
+  data.tar.gz: c5507077282d107fdab48081b9736fcfb9a9af8ca600faf316e76fc26ba16067
 SHA512:
-  metadata.gz: bc742e9c75813510b30970550c55ec34c56727e1209e2c8a71ea69c89ff1fe1a67e4abd1e4f98ae39ed5d01140f4ad04b6afd795ed264f9c3fd840c749c9ab9a
-  data.tar.gz: 20d4a84d869295af9703cc72e0c2f3f277e8735f0380337f13ea1be23c51a081c6171ae73c5c4edfee98899327490325ed4f5252513b331ff819b968e05c4066
+  metadata.gz: efb6f1688532acf22b13e6978d4b5cc65df15b2c1e781c886a701deb42206d0985d3bd1af74ddcda09377700777fb83c8278971ef567f682aee71478c278c918
+  data.tar.gz: c9a0843cf5ee0dffa774db34b254214ffc0df72bc54bda36afb4ce89650e6060f481729721f43f67bd2396b0700f61f54d92595114b9314c51205a47a5693708

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

@@ -2084,7 +2084,7 @@ EOM
                 execute_installer_cmd(master, opts)
               rescue Beaker::Host::CommandFailure => e
                 unless is_expected_pe_postgres_failure?(master)
-                  raise "Install on master failed in an unexpected manner"
+                  raise "Install on master failed in an unexpected manner: #{e.message}"
                 end
               end
             end
@@ -2132,14 +2132,23 @@ EOM
         def is_expected_pe_postgres_failure?(host)
           installer_log_dir = '/var/log/puppetlabs/installer'
           latest_installer_log_file = on(host, "ls -1t #{installer_log_dir} | head -n1").stdout.chomp
-          # As of PE Irving (PE 2018.1.x), these are the only two expected errors
+          # The initial master install pass runs before pe-postgres is fully
+          # up, so it's expected to fail: the RBAC/console database isn't
+          # reachable yet, and depending on timing any core PE service
+          # (pe-console-services, pe-puppetdb, pe-puppetserver, etc.) may
+          # not (re)start before this pass' own timeouts trip. All of these
+          # are recoverable by the later "Install/Upgrade postgres service"
+          # and "Finish install/upgrade" steps, so they shouldn't fail the
+          # test. Match the service-startup failure class generically
+          # rather than enumerating individual services/ports here.
           allowed_errors = ["The operation could not be completed because RBACs database has not been initialized",
             "Timeout waiting for the database pool to become ready",
-            "Systemd restart for pe-console-services failed",
-            "Execution of.*service pe-console-services.*: Reload timed out after 120 seconds"]
+            "Systemd (start|restart) for pe-\\S+ failed",
+            "Execution of.*service pe-console-services.*: Reload timed out after 120 seconds",
+            "Connection refused"]
 
           allowed_errors.each do |error|
-            if(on(host, "grep '#{error}' #{installer_log_dir}/#{latest_installer_log_file}", :acceptable_exit_codes => [0,1]).exit_code == 0)
+            if(on(host, "grep -E '#{error}' #{installer_log_dir}/#{latest_installer_log_file}", :acceptable_exit_codes => [0,1]).exit_code == 0)
               return true
             end
           end

data/lib/beaker-pe/version.rb

@@ -3,7 +3,7 @@ module Beaker
     module PE
 
       module Version
-        STRING = '3.6.2'
+        STRING = '3.6.3'
       end
 
     end

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

@@ -1305,76 +1305,61 @@ describe ClassMixedWithDSLInstallUtils do
   describe 'is_expected_pe_postgres_failure? method' do
     let(:mono_master) { make_host('mono_master', :pe_ver => '2017.2', :platform => 'ubuntu-16.04-x86_64', :roles => ['master', 'database', 'dashboard']) }
 
-    it 'will return true if it is the RBAC database string matcher' do
+    let(:zero_exit_code_mock) do
+      mock = Object.new
+      allow(mock).to receive(:exit_code).and_return(0)
+      mock
+    end
+    let(:one_exit_code_mock) do
+      mock = Object.new
+      allow(mock).to receive(:exit_code).and_return(1)
+      mock
+    end
+
+    def stub_installer_log_greps(all_but: nil)
       @installer_log_file_name = Beaker::Result.new( {}, '' )
       @installer_log_file_name.stdout = "installer_log_name"
-      zero_exit_code_mock = Object.new
-      allow(zero_exit_code_mock).to receive(:exit_code).and_return(0)
-      one_exit_code_mock = Object.new
-      allow(one_exit_code_mock).to receive(:exit_code).and_return(1)
       allow(subject).to receive(:on).with(mono_master, "ls -1t /var/log/puppetlabs/installer | head -n1").and_return(@installer_log_file_name)
-      allow(subject).to receive(:on).with(mono_master, "grep 'The operation could not be completed because RBACs database has not been initialized' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(zero_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Timeout waiting for the database pool to become ready' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Systemd restart for pe-console-services failed' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Execution of.*service pe-console-services.*: Reload timed out after 120 seconds' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
+
+      patterns = ["The operation could not be completed because RBACs database has not been initialized",
+        "Timeout waiting for the database pool to become ready",
+        "Systemd (start|restart) for pe-\\S+ failed",
+        "Execution of.*service pe-console-services.*: Reload timed out after 120 seconds",
+        "Connection refused"]
+
+      patterns.each do |pattern|
+        result = (pattern == all_but) ? zero_exit_code_mock : one_exit_code_mock
+        allow(subject).to receive(:on).with(mono_master, "grep -E '#{pattern}' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(result)
+      end
+    end
+
+    it 'will return true if it is the RBAC database string matcher' do
+      stub_installer_log_greps(all_but: "The operation could not be completed because RBACs database has not been initialized")
       expect(subject.is_expected_pe_postgres_failure?(mono_master)). to eq(true)
     end
 
     it 'will return true if it is the database pool timeout string matcher' do
-      @installer_log_file_name = Beaker::Result.new( {}, '' )
-      @installer_log_file_name.stdout = "installer_log_name"
-      zero_exit_code_mock = Object.new
-      allow(zero_exit_code_mock).to receive(:exit_code).and_return(0)
-      one_exit_code_mock = Object.new
-      allow(one_exit_code_mock).to receive(:exit_code).and_return(1)
-      allow(subject).to receive(:on).with(mono_master, "ls -1t /var/log/puppetlabs/installer | head -n1").and_return(@installer_log_file_name)
-      allow(subject).to receive(:on).with(mono_master, "grep 'The operation could not be completed because RBACs database has not been initialized' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Timeout waiting for the database pool to become ready' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(zero_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Systemd restart for pe-console-services failed' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Execution of.*service pe-console-services.*: Reload timed out after 120 seconds' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
+      stub_installer_log_greps(all_but: "Timeout waiting for the database pool to become ready")
       expect(subject.is_expected_pe_postgres_failure?(mono_master)). to eq(true)
     end
 
-    it 'will return true if it is the systemd restart of cosnole-services failure matcher' do
-      @installer_log_file_name = Beaker::Result.new( {}, '' )
-      @installer_log_file_name.stdout = "installer_log_name"
-      zero_exit_code_mock = Object.new
-      allow(zero_exit_code_mock).to receive(:exit_code).and_return(0)
-      one_exit_code_mock = Object.new
-      allow(one_exit_code_mock).to receive(:exit_code).and_return(1)
-      allow(subject).to receive(:on).with(mono_master, "ls -1t /var/log/puppetlabs/installer | head -n1").and_return(@installer_log_file_name)
-      allow(subject).to receive(:on).with(mono_master, "grep 'The operation could not be completed because RBACs database has not been initialized' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Timeout waiting for the database pool to become ready' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Systemd restart for pe-console-services failed' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(zero_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Execution of.*service pe-console-services.*: Reload timed out after 120 seconds' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
+    it 'will return true if it is the systemd start/restart of a pe-* service failure matcher (e.g. pe-console-services, pe-puppetdb, pe-puppetserver)' do
+      stub_installer_log_greps(all_but: "Systemd (start|restart) for pe-\\S+ failed")
       expect(subject.is_expected_pe_postgres_failure?(mono_master)). to eq(true)
     end
 
     it 'will return true if it is the console-services reload timeout string matcher' do
-      @installer_log_file_name = Beaker::Result.new( {}, '' )
-      @installer_log_file_name.stdout = "installer_log_name"
-      zero_exit_code_mock = Object.new
-      allow(zero_exit_code_mock).to receive(:exit_code).and_return(0)
-      one_exit_code_mock = Object.new
-      allow(one_exit_code_mock).to receive(:exit_code).and_return(1)
-      allow(subject).to receive(:on).with(mono_master, "ls -1t /var/log/puppetlabs/installer | head -n1").and_return(@installer_log_file_name)
-      allow(subject).to receive(:on).with(mono_master, "grep 'The operation could not be completed because RBACs database has not been initialized' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Timeout waiting for the database pool to become ready' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Systemd restart for pe-console-services failed' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Execution of.*service pe-console-services.*: Reload timed out after 120 seconds' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(zero_exit_code_mock)
+      stub_installer_log_greps(all_but: "Execution of.*service pe-console-services.*: Reload timed out after 120 seconds")
+      expect(subject.is_expected_pe_postgres_failure?(mono_master)). to eq(true)
+    end
+
+    it 'will return true if it is the connection refused matcher' do
+      stub_installer_log_greps(all_but: "Connection refused")
       expect(subject.is_expected_pe_postgres_failure?(mono_master)). to eq(true)
     end
 
     it 'will return false if no error messages are matched' do
-      @installer_log_file_name = Beaker::Result.new( {}, '' )
-      @installer_log_file_name.stdout = "installer_log_name"
-      one_exit_code_mock = Object.new
-      allow(one_exit_code_mock).to receive(:exit_code).and_return(1)
-      allow(subject).to receive(:on).with(mono_master, "ls -1t /var/log/puppetlabs/installer | head -n1").and_return(@installer_log_file_name)
-      allow(subject).to receive(:on).with(mono_master, "grep 'The operation could not be completed because RBACs database has not been initialized' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Timeout waiting for the database pool to become ready' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Systemd restart for pe-console-services failed' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
-      allow(subject).to receive(:on).with(mono_master, "grep 'Execution of.*service pe-console-services.*: Reload timed out after 120 seconds' /var/log/puppetlabs/installer/installer_log_name", :acceptable_exit_codes=>[0, 1]).and_return(one_exit_code_mock)
+      stub_installer_log_greps
       expect(subject.is_expected_pe_postgres_failure?(mono_master)). to eq(false)
     end
   end
@@ -1458,7 +1443,7 @@ describe ClassMixedWithDSLInstallUtils do
       expect(subject).to receive(:execute_installer_cmd).with(mono_master, {}).and_raise(Beaker::Host::CommandFailure).once.ordered
       allow(subject).to receive(:is_expected_pe_postgres_failure?).and_return(false)
 
-      expect{ subject.do_install_pe_with_pe_managed_external_postgres([mono_master, pe_postgres, agent], {}) }.to raise_error(RuntimeError, "Install on master failed in an unexpected manner")
+      expect{ subject.do_install_pe_with_pe_managed_external_postgres([mono_master, pe_postgres, agent], {}) }.to raise_error(RuntimeError, /\AInstall on master failed in an unexpected manner: /)
     end
 
     it 'will do a monolithic upgrade of PE with an external postgres that is managed by PE' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: beaker-pe
 version: !ruby/object:Gem::Version
-  version: 3.6.2
+  version: 3.6.3
 platform: ruby
 authors:
 - Puppetlabs
@@ -320,7 +320,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.10
+rubygems_version: 4.0.16
 specification_version: 4
 summary: Beaker PE DSL Helpers!
 test_files: []