gitlab-qa 14.9.0 → 14.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 07bcf4d830b0b1a566f938d304e7fca8060314fe8ec2880a0203a758fc50c366
-  data.tar.gz: 9c20f4c1f1c9b234f8fcc4db0f5a38527f2ca6848585bd0f218195841f7cb090
+  metadata.gz: a6f7bd80ea293183e6498166dfff6ed48a13ae11c6075fbb4a6b8da659486f0c
+  data.tar.gz: 1fb02c24469c15088ebaa23607df22dac8237c98afdf3e641bb21c9544d2a6f8
 SHA512:
-  metadata.gz: 00051bcdf47a11a19a2b106332acca427acf2c824b098bae1d5227927d88b50f5453e520033d3448e8ba2a914fdd04f34af931f19cc31e136c6d4a8bbcf334f8
-  data.tar.gz: deaf4242a1a8f543d2414c750e4f610b2e9c6e624b6bfea7a421b0946afab810a8b27bdd518e43fdf9be52f5946e1932d25544e942efc19ae7c64cc827b74074
+  metadata.gz: f3f8343857638e8ed1c2b1b496bf9f09094fddd9d14f477d7277bd71c37d967e7ecce22eb369e6a105df956a48b7ce92831812aa82356f82578bfaa6b847f1ca
+  data.tar.gz: 5ca6e86b327ad376cbae3d58184c8f0ce348f8c6830078b4fd202a16c4ec523ced8ee5e68adf89aa2c79e6a71d40f57955f43271c5fef967661b45078b214e7b

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    gitlab-qa (14.9.0)
+    gitlab-qa (14.10.0)
       activesupport (>= 6.1, < 7.2)
       gitlab (~> 4.19)
       http (~> 5.0)
@@ -175,19 +175,19 @@ GEM
     reverse_markdown (2.1.1)
       nokogiri
     rexml (3.2.5)
-    rspec (3.12.0)
-      rspec-core (~> 3.12.0)
-      rspec-expectations (~> 3.12.0)
-      rspec-mocks (~> 3.12.0)
-    rspec-core (3.12.0)
-      rspec-support (~> 3.12.0)
-    rspec-expectations (3.12.0)
+    rspec (3.13.0)
+      rspec-core (~> 3.13.0)
+      rspec-expectations (~> 3.13.0)
+      rspec-mocks (~> 3.13.0)
+    rspec-core (3.13.0)
+      rspec-support (~> 3.13.0)
+    rspec-expectations (3.13.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.12.0)
-    rspec-mocks (3.12.0)
+      rspec-support (~> 3.13.0)
+    rspec-mocks (3.13.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.12.0)
-    rspec-support (3.12.0)
+      rspec-support (~> 3.13.0)
+    rspec-support (3.13.1)
     rubocop (1.43.0)
       json (~> 2.3)
       parallel (~> 1.10)
@@ -280,7 +280,7 @@ DEPENDENCIES
   pry (~> 0.11)
   pry-byebug (~> 3.10.1)
   rake (~> 13.0)
-  rspec (~> 3.7)
+  rspec (~> 3.13)
   simplecov (~> 0.22)
   simplecov-cobertura (~> 2.1)
   solargraph (~> 0.41)

data/gitlab-qa.gemspec

@@ -29,7 +29,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'pry', '~> 0.11'
   spec.add_development_dependency 'pry-byebug', '~> 3.10.1'
   spec.add_development_dependency 'rake', '~> 13.0'
-  spec.add_development_dependency 'rspec', '~> 3.7'
+  spec.add_development_dependency 'rspec', '~> 3.13'
   spec.add_development_dependency 'simplecov', '~> 0.22'
   spec.add_development_dependency 'simplecov-cobertura', '~> 2.1'
   spec.add_development_dependency 'solargraph', '~> 0.41'

data/lib/gitlab/qa/component/base.rb

@@ -189,7 +189,7 @@ module Gitlab
           raise 'Invalid instance name!' unless name
         end
 
-        def instance_no_teardown
+        def instance_no_teardown # rubocop:disable Metrics/AbcSize
           begin
             retries ||= 0
             prepare
@@ -201,12 +201,17 @@ module Gitlab
             reconfigure_log_file = get_reconfigure_log_file_from_artefact
             # for scenarios where a service fails during startup, attempt to retry to avoid flaky failures
             if (retries += 1) < 3
-              Runtime::Logger.info(
-                "Follow the document " \
-                "https://gitlab.com/gitlab-org/quality/runbooks/-/blob/main/debug_orchestrated_test_locally/ " \
-                "for debugging the test failure locally.")
-              Runtime::Logger.error("Tailing the reconfigure log file: #{reconfigure_log_file}. Please check #{reconfigure_log_file} " \
-                                    "file in the artifacts for more details.")
+              unless reconfigure_log_file.nil?
+                Runtime::Logger.info(
+                  "Follow the document " \
+                  "https://gitlab.com/gitlab-org/quality/runbooks/-/blob/main/debug_orchestrated_test_locally/ " \
+                  "for debugging the test failure locally.")
+
+                # Tailing the reconfigure logs after retries are over and before raising exception
+                Runtime::Logger.info("Tail of the reconfigure log file, see artifacts for full log: #{reconfigure_log_file}")
+                Support::ShellCommand.new("tail -n 100 #{reconfigure_log_file}", stream_output: true).execute!
+              end
+
               Runtime::Logger.warn(
                 "Retry instance_no_teardown due to Support::ShellCommand::StatusError -- attempt #{retries}"
               )
@@ -214,9 +219,14 @@ module Gitlab
               retry
             end
 
-            # Tailing the reconfigure logs after retries are over and before raising exception
-            Runtime::Logger.info("Tail of the reconfigure log file, see artifacts for full log: #{reconfigure_log_file}")
-            Support::ShellCommand.new("tail -n 100 #{reconfigure_log_file}", stream_output: true).execute!
+            # Printing logs to stdout for last retry failure
+
+            if !reconfigure_log_file.nil? && retries == 3
+              # Tailing the reconfigure logs after retries are over and before raising exception
+              Runtime::Logger.info("Tail of the reconfigure log file, see artifacts for full log: #{reconfigure_log_file}")
+              Support::ShellCommand.new("tail -n 100 #{reconfigure_log_file}", stream_output: true).execute!
+            end
+
             raise e
           end
 

data/lib/gitlab/qa/component/gitaly.rb

@@ -67,7 +67,8 @@ module Gitlab
             gitaly['configuration'] = {
               'listen_addr': '0.0.0.0:#{gitaly_port}',
               'prometheus_listen_addr': '0.0.0.0:9236',
-              'auth_token': 'PRAEFECT_INTERNAL_TOKEN'
+              'auth_token': 'PRAEFECT_INTERNAL_TOKEN',
+              'transactions': {'enabled': #{Runtime::Env.qa_gitaly_transactions_enabled?}}
             }
             gitlab_shell['secret_token'] = 'GITLAB_SHELL_SECRET_TOKEN';
             gitlab_rails['internal_api_url'] = 'http://#{cluster_config.gitlab_name}.#{cluster_config.network}';

data/lib/gitlab/qa/runtime/env.rb

@@ -370,7 +370,8 @@ module Gitlab
         end
 
         def video_recorder_image
-          env_var_value_if_defined('QA_VIDEO_RECORDER_IMAGE') || 'registry.gitlab.com/gitlab-org/gitlab-qa/selenoid-manual-video-recorder'
+          env_var_value_if_defined('QA_VIDEO_RECORDER_IMAGE') ||
+            'registry.gitlab.com/gitlab-org/gitlab-qa/selenoid-manual-video-recorder'
         end
 
         def video_recorder_version
@@ -470,6 +471,10 @@ module Gitlab
           enabled?(env_var_value_if_defined('QA_RETRY_FAILED_SPECS'), default: false)
         end
 
+        def self.qa_gitaly_transactions_enabled?
+          enabled?(env_var_value_if_defined('QA_GITALY_TRANSACTIONS_ENABLED'), default: false)
+        end
+
         private
 
         def enabled?(value, default: true)

data/lib/gitlab/qa/scenario/test/instance/airgapped.rb

@@ -37,7 +37,7 @@ module Gitlab
                 gitlab.skip_availability_check = true
                 gitlab.omnibus_configuration << gitlab_omnibus_configuration
                 rspec_args << "--" unless rspec_args.include?('--')
-                rspec_args << %w[--tag ~orchestrated]
+                rspec_args << "--tag ~orchestrated"
                 gitlab.instance do
                   Component::Specs.perform do |specs|
                     specs.suite = 'Test::Instance::Airgapped'

data/lib/gitlab/qa/scenario/test/instance/repository_storage.rb

@@ -22,7 +22,7 @@ module Gitlab
                   cluster.join
 
                   rspec_args << "--" unless rspec_args.include?('--')
-                  rspec_args << %w[--tag repository_storage]
+                  rspec_args << "--tag repository_storage]"
 
                   Component::Specs.perform do |specs|
                     specs.suite = 'Test::Instance::All'

data/lib/gitlab/qa/scenario/test/integration/integrations.rb

@@ -14,7 +14,7 @@ module Gitlab
                 gitlab.instance do
                   Component::Specs.perform do |specs|
                     rspec_args << '--' unless rspec_args.include?('--')
-                    rspec_args << %w[--tag integrations]
+                    rspec_args << "--tag integrations"
                     specs.suite = 'Test::Instance::All'
                     specs.hostname = "qa-e2e-specs.#{gitlab.network}"
                     specs.release = gitlab.release

data/lib/gitlab/qa/scenario/test/integration/jira.rb

@@ -21,7 +21,7 @@ module Gitlab
                       Runtime::Logger.info('Running Jira specs!')
 
                       rspec_args << "--" unless rspec_args.include?('--')
-                      rspec_args << %w[--tag jira]
+                      rspec_args << "--tag jira"
 
                       Component::Specs.perform do |specs|
                         specs.suite = 'Test::Instance::All'

data/lib/gitlab/qa/version.rb

@@ -2,6 +2,6 @@
 
 module Gitlab
   module QA
-    VERSION = '14.9.0'
+    VERSION = '14.10.0'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-qa
 version: !ruby/object:Gem::Version
-  version: 14.9.0
+  version: 14.10.0
 platform: ruby
 authors:
 - GitLab Quality
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-05-02 00:00:00.000000000 Z
+date: 2024-05-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: climate_control
@@ -114,14 +114,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.7'
+        version: '3.13'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.7'
+        version: '3.13'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement