gitlab-qa 5.16.0 → 6.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bc79855be61afd8dfaef793cb1a6520bde1f07f6c44a18b9fdd605e26ac947c4
-  data.tar.gz: d326f64befb861020097fd55b9772b54d7014509e87c5c13c71e619c11a21a65
+  metadata.gz: 6ae53aaf2bbb205e8b04452a569bf5eac0c6743f55455f0f7897e36f0aa5e69f
+  data.tar.gz: 0b0a56d2d6e5afde89ba9f0f76db8dd1b8c0fb288d727fa6343423d4d9d33a73
 SHA512:
-  metadata.gz: e497734ed2d4d9d58c2dcdf051eb3a4a6ad02adcfc7954ae54ac83ee912b47ff0d6f8478ea6e272a8a93feb14cdda8d55632fb0f92e97379596a12b67ee25f57
-  data.tar.gz: fc927589ca591708a7b61341acf69d48125812642ce9ca87f0183cf11bcf14c55d6bfa8bb04fe1cf3c4685e1d4da0781bab6af15a942c0675ae149fac8b20704
+  metadata.gz: bd2a3ef1881f63da084930753c9d093c50c130c1ffa6923c5b91e2c9c98cbac48c84e825d350ed35b2d75f92704311eef249ee87cbff0d80c3ecdd3a9b426144
+  data.tar.gz: '0678f05968f62afb7f1266aec93824f746010a2155c2ee50a11588f5d4e3e07ae6b88dea59b5bb67c85e722a67cc543ee49faf7e433feba6dedd2dc55dd16b24'

data/.gitlab-ci.yml

@@ -94,7 +94,7 @@ release:
     - exe/gitlab-qa ${QA_SCENARIO:=Test::Instance::Image} ${RELEASE:=$DEFAULT_RELEASE} -- $QA_TESTS $QA_RSPEC_TAGS $RSPEC_REPORT_OPTS || test_run_exit_code=$?
     - exe/gitlab-qa-report --update-screenshot-path "gitlab-qa-run-*/**/rspec-*.xml"
     - export GITLAB_QA_ACCESS_TOKEN="$GITLAB_QA_PRODUCTION_ACCESS_TOKEN"
-    - if [ "$TOP_UPSTREAM_SOURCE_REF" == "master" ]; then exe/gitlab-qa-report --report-in-issues "gitlab-qa-run-*/**/rspec-*.xml" --project "$QA_TESTCASES_REPORTING_PROJECT"; fi
+    - if [ "$TOP_UPSTREAM_SOURCE_REF" == "master" ]; then exe/gitlab-qa-report --report-in-issues "gitlab-qa-run-*/**/rspec-*.xml" --project "$QA_TESTCASES_REPORTING_PROJECT" || true; fi
     - exit $test_run_exit_code
 
 .ce-qa:
@@ -143,7 +143,7 @@ release:
 .rspec-report-opts:
   variables:
     FILE_SAFE_JOB_NAME: $(echo $CI_JOB_NAME | sed 's/[ /]/_/g')
-    RSPEC_REPORT_OPTS: "--format RspecJunitFormatter --out \"tmp/rspec-${CI_JOB_ID}.xml\" --format html --out \"tmp/rspec-${FILE_SAFE_JOB_NAME}.htm\" --color --format documentation"
+    RSPEC_REPORT_OPTS: "--format QA::Support::JsonFormatter --out \"tmp/rspec-${CI_JOB_ID}.json\" --format RspecJunitFormatter --out \"tmp/rspec-${CI_JOB_ID}.xml\" --format html --out \"tmp/rspec-${FILE_SAFE_JOB_NAME}.htm\" --color --format documentation"
 
 .quarantine:
   allow_failure: true

data/docs/what_tests_can_be_run.md

@@ -1,5 +1,6 @@
-# What tests can be run?
 
+# What tests can be run?
+[[_TOC_]]
 ## The two types of QA tests
 
 First of all, the first thing to choose is whether you want to run orchestrated
@@ -28,6 +29,8 @@ For more details on the internals, please read the
 
 ## Supported GitLab environment variables
 
+All environment variables used by GitLab QA should be defined in [`lib/gitlab/qa/runtime/env.rb`](https://gitlab.com/gitlab-org/gitlab-qa/-/blob/master/lib/gitlab/qa/runtime/env.rb).
+
 |        Variable       | Default   | Description           | Required |
 |-----------------------|-----------|-----------------------|----------|
 | `GITLAB_USERNAME`       | `root`  | Username to use when signing into GitLab. | Yes|
@@ -53,7 +56,8 @@ For more details on the internals, please read the
 | `GITLAB_SANDBOX_NAME`   | `gitlab-qa-sandbox` | The sandbox group name the test suite is going to use. | No|
 | `GITLAB_QA_ACCESS_TOKEN`|-  | A valid personal access token with the `api` scope. This is used for API access during tests, and is used in the version that staging is currently running. An existing token that is valid on [`Test::Instance::Staging`](#testinstancestaging) scenario to retrieve the staging can be found in the shared 1Password vault. |No|
 | `GITLAB_QA_ADMIN_ACCESS_TOKEN` |-  | A valid personal access token with the `api` scope from a user with admin access. Used for API access as an admin during tests. | No|
-| `EE_LICESEN` |-  | Enterprise Edition license. | No|
+| `GITLAB_QA_CONTAINER_REGISTRY_ACCESS_TOKEN` | - | A valid personal access token with the `read_registry` scope. Used to [access the container registry on `registry.gitlab.com` when tests run in a CI job that _is not_ triggered via another pipeline](https://gitlab.com/gitlab-org/gitlab-qa/-/blob/364addb83e7b136ff0f9d8719ca9553d290aa9ab/lib/gitlab/qa/release.rb#L152). For example, if you manually run a [new Staging pipeline](https://ops.gitlab.net/gitlab-org/quality/staging/-/pipelines/new), this token will be used. | No |
+| `EE_LICENSE` |-  | Enterprise Edition license. | No|
 | `QA_ARTIFACTS_DIR` |`/tmp/gitlab-qa`| Path to a directory where artifacts (logs and screenshots) for failing tests will be saved. | No|
 | `DOCKER_HOST` |`http://localhost`| Docker host to run tests against. | No|
 | `CHROME_HEADLESS` |-  | When running locally, set to `false` to allow Chrome tests to be visible - watch your tests being run. | No|
@@ -78,13 +82,33 @@ For more details on the internals, please read the
 | `GEO_MAX_DB_REPLICATION_TIME` | `120` | Maximum time that a test will wait for database data to appear on a Geo secondary node. | No|
 | `JIRA_ADMIN_USERNAME` |-  | Username for authenticating with Jira server as admin. | No|
 | `JIRA_ADMIN_PASSWORD` |-  | Password for authenticating with Jira server as admin. | No|
+| `CACHE_NAMESPACE_NAME` | `true` | Cache namespace name for groups. | No|
 
 ## [Supported Remote Grid environment variables](./running_against_remote_grid.md)
 
 ## Running tests with a feature flag enabled
 
-It is possible to enable a feature flag before running tests. See the [QA
-framework documentation](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/qa/README.md#running-tests-with-a-feature-flag-enabled) for details.
+It is possible to enable or disable a feature flag before running tests.
+To test a Gitlab image with a feature flag enabled, run this command:
+```
+$ gitlab-qa Test::Instance::Image gitlab/gitlab-ee:12.4.0-ee.0 --enable-feature feature_flag_name
+```
+
+To run a test with feature flag disabled, run this command:
+```
+$ gitlab-qa Test::Instance::Image gitlab/gitlab-ee:12.4.0-ee.0 --disable-feature feature_flag_name
+```
+
+You an also test a GitLab image multiple times with different feature flag settings:
+```
+$ gitlab-qa Test::Instance::Image gitlab/gitlab-ee:12.4.0-ee.0 --disable-feature feature_flag_name --enable-feature feature_flag_name
+```
+This will first disable `feature_flag_name` flag and run the tests and then enable it and run the tests again.
+
+You can pass any number of feature flag settings. The tests will run once for each setting.
+
+See the [QA framework documentation](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/qa/README.md#running-tests-with-a-feature-flag-enabled)
+for information on running the tests with different feature flag settings from the QA framework.
 
 ## Specifying the GitLab version
 
@@ -112,6 +136,10 @@ For example, the following command would use the image named `gitlab/gitlab-ee:n
 $ gitlab-qa Test::Instance::Image EE
 ```
 
+To run EE tests, the `EE_LICENSE` environment variable needs to be set:
+
+`$ export EE_LICENSE=$(cat /path/to/GitLab.gitlab_license)`
+
 ## Examples
 
 ### `Test::Instance::Image CE|EE|<full image address>`
@@ -217,6 +245,25 @@ $ gitlab-qa Test::Integration::Geo EE
 
 [test-geo]: https://gitlab.com/gitlab-org/gitlab-ee/blob/master/qa/qa/ee/scenario/test/geo.rb
 
+### `Test::Integration::GitalyCluster CE|EE|<full image address>`
+
+This tests [Gitaly Cluster](https://docs.gitlab.com/ee/administration/gitaly/praefect.html),
+a clustered configuration of the Gitaly repository storage service.
+
+The scenario configures and starts several docker containers to
+represent the [recommended minimum configuration](https://docs.gitlab.com/ee/administration/gitaly/praefect.html#requirements-for-configuring-a-gitaly-cluster)
+of a Gitaly Cluster.
+
+To run tests against the GitLab container, a GitLab QA (`gitlab/gitlab-qa`)
+container is spun up and tests are run from it by running the
+`Test::Instance::All` scenario with the `:gitaly_cluster` tag.
+
+Example:
+
+```
+$ gitlab-qa Test::Integration::GitalyCluster EE
+```
+
 ### `Test::Integration::LDAPNoTLS CE|EE|<full image address>`
 
 This tests that a GitLab instance works as expected with an external
@@ -545,7 +592,7 @@ in the GitLab CE project).
 
 - `GITLAB_QA_DEV_ACCESS_TOKEN`: A valid personal access token for the
   `gitlab-qa-bot` on `dev.gitlab.org` with the `registry` scope.
-  This is used to pull the QA Docker from the Omnibus GitLab `dev` Container Registry.
+  This is used to pull the QA Docker image from the Omnibus GitLab `dev` Container Registry.
   If the variable isn't present, the QA image from Docker Hub will be used.
   This can be found in the shared 1Password vault.
 
@@ -560,6 +607,41 @@ $ export GITLAB_PASSWORD="$GITLAB_QA_PASSWORD"
 $ gitlab-qa Test::Instance::Staging
 ```
 
+### `Test::Instance::StagingGeo`
+
+This scenario tests that the Geo staging deployment (with [`staging.gitlab.com`](https://staging.gitlab.com) as the primary site and [`geo.staging.gitlab.com`](https://geo.staging.gitlab.com) as the secondary site) works as expected by running tests tagged `:geo` against it. This is done by spinning up a GitLab QA (`gitlab/gitlab-qa`) container and running the `QA::EE::Scenario::Test::Geo` scenario. Note that the Geo setup steps in the `QA::EE::Scenario::Test::Geo` scenario are skipped when testing a live Geo deployment.
+
+**Required user properties:**
+
+- The user must provide OAuth authorization on the secondary site before running Geo tests. This can be done via the authorization modal that appears after logging into the secondary node for the first time.
+
+- Some Geo tests require the user to have Admin access level (for example, the Geo Nodes API tests)
+
+**Required environment variables:**
+
+- `GITLAB_QA_ACCESS_TOKEN`: A valid personal access token with the `api` scope.
+  This is used to retrieve the version that staging is currently running.
+  This can be found in the shared 1Password vault.
+
+**Optional environment variables:**
+
+- `GITLAB_QA_DEV_ACCESS_TOKEN`: A valid personal access token for the
+  `gitlab-qa-bot` on `dev.gitlab.org` with the `registry` scope.
+  This is used to pull the QA Docker image from the Omnibus GitLab `dev` Container Registry.
+  If the variable isn't present, the QA image from Docker Hub will be used.
+  This can be found in the shared 1Password vault.
+
+Example:
+
+```
+$ export GITLAB_QA_ACCESS_TOKEN=your_api_access_token
+$ export GITLAB_QA_DEV_ACCESS_TOKEN=your_dev_registry_access_token
+$ export GITLAB_USERNAME="gitlab-qa"
+$ export GITLAB_PASSWORD="$GITLAB_QA_PASSWORD"
+
+$ gitlab-qa Test::Instance::StagingGeo
+```
+
 ### `Test::Instance::Production`
 
 This scenario functions the same as `Test::Instance::Staging`
@@ -633,6 +715,25 @@ $ export EE_LICENSE=$(cat /path/to/gitlab_license)
 $ gitlab-qa Test::Instance::Airgapped EE -- --tag smoke
 ```
 
+### `Test::Instance::Geo EE|<full image address>:nightly|latest|any_tag http://geo-primary.gitlab http://geo-secondary.gitlab`
+
+This scenario will run tests tagged `:geo` against a live Geo deployment, by spinning up a GitLab QA (`gitlab/gitlab-qa`)
+container and running the `QA::EE::Scenario::Test::Geo` scenario. Note that the Geo setup steps in the `QA::EE::Scenario::Test::Geo` scenario are skipped when testing a live Geo deployment. The URLs for the primary site and secondary site of the live Geo deployment must be provided as command line arguments.
+
+**Required user properties:**
+
+- The user must provide OAuth authorization on the secondary site before running Geo tests. This can be done via the authorization modal that appears after signing into the secondary node for the first time.
+
+- Some Geo tests require the user to have Admin access level (for example, the Geo Nodes API tests)
+
+Example:
+
+```
+$ export GITLAB_USERNAME="gitlab-qa"
+$ export GITLAB_PASSWORD="$GITLAB_QA_PASSWORD"
+
+$ gitlab-qa Test::Instance::Geo EE https://primary.gitlab.com https://secondary.gitlab.com
+```
 ----
 
 [Back to README.md](../README.md)

data/lib/gitlab/qa.rb

@@ -97,9 +97,12 @@ module Gitlab
     end
 
     module Report
+      autoload :JsonTestResults, 'gitlab/qa/report/json_test_results'
+      autoload :JUnitTestResults, 'gitlab/qa/report/junit_test_results'
       autoload :PrepareStageReports, 'gitlab/qa/report/prepare_stage_reports'
       autoload :ResultsInIssues, 'gitlab/qa/report/results_in_issues'
       autoload :SummaryTable, 'gitlab/qa/report/summary_table'
+      autoload :TestResult, 'gitlab/qa/report/test_result'
       autoload :UpdateScreenshotPath, 'gitlab/qa/report/update_screenshot_path'
     end
 

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

@@ -41,26 +41,16 @@ module Gitlab
           raise NotImplementedError, "#{self.class.name} must specify a docker image tag as DOCKER_IMAGE_TAG"
         end
 
-        def instance
+        def instance(skip_teardown: false)
           instance_no_teardown do
             yield self if block_given?
           end
         ensure
-          teardown
+          teardown unless skip_teardown
         end
 
         alias_method :launch_and_teardown_instance, :instance
 
-        def instance_no_teardown
-          prepare
-          start
-          reconfigure
-          wait_until_ready
-          process_exec_commands
-
-          yield self if block_given?
-        end
-
         def prepare
           prepare_docker_image
           prepare_network
@@ -147,6 +137,16 @@ module Gitlab
           raise 'Invalid instance name!' unless name
         end
 
+        def instance_no_teardown
+          prepare
+          start
+          reconfigure
+          wait_until_ready
+          process_exec_commands
+
+          yield self if block_given?
+        end
+
         def teardown?
           !Runtime::Scenario.attributes.include?(:teardown) || Runtime::Scenario.teardown
         end

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

@@ -27,26 +27,50 @@ module Gitlab
 
           name = "#{release.project_name}-qa-#{SecureRandom.hex(4)}"
 
-          @docker.run(release.qa_image, release.qa_tag, suite, *args) do |command|
-            command << "-t --rm --net=#{network || 'bridge'}"
+          feature_flag_sets = []
 
-            env.merge(Runtime::Env.variables).each do |key, value|
-              command.env(key, value)
-            end
+          # When `args` includes `[..., "--disable-feature", "a", "--enable-feature", "b", ...]`
+          # `feature_flag_sets` will be set to `[["--disable-feature", "a"], ["--enable-feature", "b"]]`
+          # This will result in tests running twice, once with each feature.
+          while (index = args&.index { |x| x =~ /--.*-feature/ })
+            feature_flag_sets << args.slice!(index, 2)
+          end
 
-            command.volume('/var/run/docker.sock', '/var/run/docker.sock')
-            command.volume(File.join(Runtime::Env.host_artifacts_dir, name), File.join(Docker::Volumes::QA_CONTAINER_WORKDIR, 'tmp'))
+          # When `args` do not have either "--disable-feature" or "--enable-feature", we
+          # add [] so that test is run exactly once.
+          feature_flag_sets << [] unless feature_flag_sets.any?
 
-            @volumes.to_h.each do |to, from|
-              command.volume(to, from)
-            end
+          feature_flag_sets.each do |feature_flag_set|
+            @docker.run(release.qa_image, release.qa_tag, suite, *args_with_flags(args, feature_flag_set)) do |command|
+              command << "-t --rm --net=#{network || 'bridge'}"
+
+              env.merge(Runtime::Env.variables).each do |key, value|
+                command.env(key, value)
+              end
+
+              command.volume('/var/run/docker.sock', '/var/run/docker.sock')
+              command.volume(File.join(Runtime::Env.host_artifacts_dir, name), File.join(Docker::Volumes::QA_CONTAINER_WORKDIR, 'tmp'))
+
+              @volumes.to_h.each do |to, from|
+                command.volume(to, from)
+              end
 
-            command.name(name)
+              command.name(name)
+            end
           end
         end
 
         private
 
+        def args_with_flags(args, feature_flag_set)
+          return args if feature_flag_set.empty?
+
+          puts "Running with feature flag: #{feature_flag_set.join(' ')}"
+
+          args_with_f = args.dup
+          args_with_f.insert(1, *feature_flag_set)
+        end
+
         def skip_tests?
           Runtime::Scenario.attributes.include?(:run_tests) && !Runtime::Scenario.run_tests
         end

data/lib/gitlab/qa/report/json_test_results.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+require 'json'
+
+module Gitlab
+  module QA
+    module Report
+      class JsonTestResults
+        include Enumerable
+
+        def initialize(file)
+          @testcases = JSON.parse(File.read(file))['examples'].map { |test| TestResult.from_json(test) }
+        end
+
+        def each(&block)
+          @testcases.each(&block)
+        end
+      end
+    end
+  end
+end

data/lib/gitlab/qa/report/junit_test_results.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+require 'nokogiri'
+
+module Gitlab
+  module QA
+    module Report
+      class JUnitTestResults
+        include Enumerable
+
+        def initialize(file)
+          @testcases = Nokogiri::XML(File.read(file)).xpath('//testcase').map { |test| TestResult.from_junit(test) }
+        end
+
+        def each(&block)
+          @testcases.each(&block)
+        end
+      end
+    end
+  end
+end

data/lib/gitlab/qa/report/results_in_issues.rb

@@ -46,7 +46,18 @@ module Gitlab
 
           Dir.glob(files).each do |file|
             puts "Reporting tests in #{file}"
-            Nokogiri::XML(File.open(file)).xpath('//testcase').each do |test|
+            extension = File.extname(file)
+
+            case extension
+            when '.json'
+              test_results = Report::JsonTestResults.new(file)
+            when '.xml'
+              test_results = Report::JUnitTestResults.new(file)
+            else
+              raise "Unknown extension #{extension}"
+            end
+
+            test_results.each do |test|
               report_test(test)
             end
           end
@@ -99,9 +110,9 @@ module Gitlab
         end
 
         def report_test(test)
-          return if test.search('skipped').any?
+          return if test.skipped
 
-          puts "Reporting test: #{test['file']} | #{test['name']}"
+          puts "Reporting test: #{test.file} | #{test.name}"
 
           issue = find_issue(test)
           if issue
@@ -124,29 +135,37 @@ module Gitlab
             Gitlab.create_issue(
               project,
               title_from_test(test),
-              { description: "### Full description\n\n#{search_safe(test['name'])}\n\n### File path\n\n#{test['file']}", labels: 'status::automated' }
+              { description: "### Full description\n\n#{search_safe(test.name)}\n\n### File path\n\n#{test.file}", labels: 'status::automated' }
             )
           end
         end
 
+        # rubocop:disable Metrics/AbcSize
         def find_issue(test)
           handle_gitlab_client_exceptions do
+            return Gitlab.issue(project, id_from_status_issue_url(test.status_issue)) if test.status_issue
+
             issues = Gitlab.issues(project, { search: search_term(test) })
               .auto_paginate
               .select { |issue| issue.state == 'opened' && issue.title.strip == title_from_test(test) }
 
-            warn(%(Too many issues found with the file path "#{test['file']}" and name "#{test['name']}")) if issues.many?
+            warn(%(Too many issues found with the file path "#{test.file}" and name "#{test.name}")) if issues.many?
 
             issues.first
           end
         end
+        # rubocop:enable Metrics/AbcSize
+
+        def id_from_status_issue_url(url)
+          url.split('/').last.to_i
+        end
 
         def search_term(test)
-          %("#{test['file']}" "#{search_safe(test['name'])}")
+          %("#{test.file}" "#{search_safe(test.name)}")
         end
 
         def title_from_test(test)
-          title = "#{partial_file_path(test['file'])} | #{search_safe(test['name'])}".strip
+          title = "#{partial_file_path(test.file)} | #{search_safe(test.name)}".strip
 
           return title unless title.length > MAX_TITLE_LENGTH
 
@@ -162,7 +181,7 @@ module Gitlab
         end
 
         def note_status(issue, test)
-          return if failures(test).empty?
+          return if test.failures.empty?
 
           note = note_content(test)
 
@@ -176,7 +195,7 @@ module Gitlab
         end
 
         def note_content(test)
-          errors = failures(test).each_with_object([]) do |failure, text|
+          errors = test.failures.each_with_object([]) do |failure, text|
             text << <<~TEXT
               Error:
               ```
@@ -185,7 +204,7 @@ module Gitlab
 
               Stacktrace:
               ```
-              #{failure.content}
+              #{failure['stacktrace']}
               ```
             TEXT
           end.join("\n\n")
@@ -231,7 +250,7 @@ module Gitlab
         def update_labels(issue, test)
           labels = issue.labels
           labels.delete_if { |label| label.start_with?("#{pipeline}::") }
-          labels << (failures(test).empty? ? "#{pipeline}::passed" : "#{pipeline}::failed")
+          labels << (test.failures.empty? ? "#{pipeline}::passed" : "#{pipeline}::failed")
           labels << "Enterprise Edition" if ee_test?(test)
           quarantine_job? ? labels << "quarantine" : labels.delete("quarantine")
 
@@ -242,11 +261,7 @@ module Gitlab
         # rubocop:enable Metrics/AbcSize
 
         def ee_test?(test)
-          test['file'] =~ %r{features/ee/(api|browser_ui)}
-        end
-
-        def failures(test)
-          test.search('failure')
+          test.file =~ %r{features/ee/(api|browser_ui)}
         end
 
         def pipeline

data/lib/gitlab/qa/report/test_result.rb

@@ -0,0 +1,64 @@
+# frozen_string_literal: true
+
+module Gitlab
+  module QA
+    module Report
+      class TestResult
+        attr_accessor :name, :file, :skipped, :failures, :status_issue
+
+        def self.from_json(test)
+          new.tap do |test_result|
+            test_result.name = test['full_description']
+            test_result.file = test['file_path']
+            test_result.skipped = test['status'] == 'pending'
+            test_result.failures = failures_from_json_exceptions(test)
+            test_result.status_issue = test['status_issue']
+          end
+        end
+
+        def self.from_junit(test)
+          new.tap do |test_result|
+            test_result.name = test['name']
+            test_result.file = test['file']
+            test_result.skipped = test.search('skipped').any?
+            test_result.failures = failures_from_junit_exceptions(test)
+          end
+        end
+
+        def self.failures_from_json_exceptions(test)
+          return [] unless test.key?('exceptions')
+
+          test['exceptions'].map do |exception|
+            spec_file_first_index = exception['backtrace'].rindex do |line|
+              line.include?(File.basename(test['file_path']))
+            end
+
+            {
+              'message' => "#{exception['class']}: #{exception['message']}",
+              'stacktrace' => exception['backtrace'].slice(0..spec_file_first_index).join("\n")
+            }
+          end
+        end
+        private_class_method :failures_from_json_exceptions
+
+        def self.failures_from_junit_exceptions(test)
+          failures = test.search('failure')
+          return [] if failures.empty?
+
+          failures.map do |exception|
+            trace = exception.content.split("\n").map(&:strip)
+            spec_file_first_index = trace.rindex do |line|
+              line.include?(File.basename(test['file']))
+            end
+
+            {
+              'message' => "#{exception['type']}: #{exception['message']}",
+              'stacktrace' => trace.slice(0..spec_file_first_index).join("\n")
+            }
+          end
+        end
+        private_class_method :failures_from_junit_exceptions
+      end
+    end
+  end
+end

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

@@ -6,6 +6,8 @@ module Gitlab
       module Env
         extend self
 
+        # Variables that are used in tests and are passed through to the docker container that executes the tests.
+        # These variables should be listed in /docs/what_tests_can_be_run.md#supported-gitlab-environment-variables
         ENV_VARIABLES = {
           'QA_REMOTE_GRID' => :remote_grid,
           'QA_REMOTE_GRID_USERNAME' => :remote_grid_username,
@@ -78,7 +80,8 @@ module Gitlab
           'GEO_MAX_DB_REPLICATION_TIME' => :geo_max_db_replication_time,
           'JIRA_HOSTNAME' => :jira_hostname,
           'JIRA_ADMIN_USERNAME' => :jira_admin_username,
-          'JIRA_ADMIN_PASSWORD' => :jira_admin_password
+          'JIRA_ADMIN_PASSWORD' => :jira_admin_password,
+          'CACHE_NAMESPACE_NAME' => :cache_namespace_name
         }.freeze
 
         ENV_VARIABLES.each_value do |accessor|

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

@@ -8,9 +8,6 @@ module Gitlab
 
           class Geo < Scenario::Template
             def perform(release, primary_address, secondary_address, *rspec_args)
-              # Geo requires an EE license
-              Runtime::Env.require_license!
-
               Component::Specs.perform do |specs|
                 specs.suite = 'QA::EE::Scenario::Test::Geo'
                 specs.release = QA::Release.new(release)

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

@@ -7,13 +7,13 @@ module Gitlab
             attr_reader :gitlab_name, :spec_suite
 
             def initialize
-              @gitlab_name = 'gitlab-gitaly-ha'
+              @gitlab_name = 'gitlab-gitaly-cluster'
               @primary_node_name = 'gitaly1'
               @secondary_node_name = 'gitaly2'
               @tertiary_node_name = 'gitaly3'
               @praefect_node_name = 'praefect'
               @database = 'postgres'
-              @spec_suite = 'Test::Integration::GitalyHA'
+              @spec_suite = 'Test::Instance::All'
               @network = 'test'
             end
 
@@ -26,7 +26,7 @@ module Gitlab
               sql_node = Component::PostgreSQL.new.tap do |sql|
                 sql.name = @database
                 sql.network = @network
-                sql.instance_no_teardown do
+                sql.instance(skip_teardown: true) do
                   sql.run_psql '-d template1 -c "CREATE DATABASE praefect_production OWNER postgres"'
                 end
               end
@@ -39,7 +39,7 @@ module Gitlab
 
                 praefect.omnibus_config = praefect_omnibus_configuration
 
-                praefect.instance_no_teardown
+                praefect.instance(skip_teardown: true)
               end
 
               Component::Gitlab.perform do |gitlab|
@@ -49,7 +49,10 @@ module Gitlab
 
                 gitlab.omnibus_config = gitlab_omnibus_configuration
                 gitlab.instance do
-                  puts "Running Gitaly HA specs!"
+                  puts "Running Gitaly Cluster specs!"
+
+                  rspec_args << "--" unless rspec_args.include?('--')
+                  rspec_args << %w[--tag gitaly_cluster]
 
                   Component::Specs.perform do |specs|
                     specs.suite = spec_suite
@@ -190,7 +193,7 @@ module Gitlab
                 gitaly.network = @network
                 gitaly.skip_availability_check = true
                 gitaly.omnibus_config = gitaly_omnibus_configuration
-                gitaly.instance_no_teardown
+                gitaly.instance(skip_teardown: true)
               end
             end
           end

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

@@ -65,6 +65,7 @@ module Gitlab
             def omnibus_config_with_praefect
               <<~OMNIBUS
                 gitaly['enable'] = true;
+                gitaly['listen_addr'] = '0.0.0.0:8075';
                 gitaly['auth_token'] = 'secret-token';
                 gitaly['storage'] = [
                   {
@@ -82,7 +83,7 @@ module Gitlab
                 praefect['virtual_storages'] = {
                   'default' => {
                     'praefect-gitaly-0' => {
-                      'address' => 'unix:/var/opt/gitlab/gitaly/gitaly.socket',
+                      'address' => 'tcp://localhost:8075',
                       'token'   => 'secret-token',
                       'primary' => true
                     }

data/lib/gitlab/qa/version.rb

@@ -1,5 +1,5 @@
 module Gitlab
   module QA
-    VERSION = '5.16.0'.freeze
+    VERSION = '6.1.1'.freeze
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-qa
 version: !ruby/object:Gem::Version
-  version: 5.16.0
+  version: 6.1.1
 platform: ruby
 authors:
 - Grzegorz Bizon
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-07-14 00:00:00.000000000 Z
+date: 2020-08-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: climate_control
@@ -257,9 +257,12 @@ files:
 - lib/gitlab/qa/docker/shellout.rb
 - lib/gitlab/qa/docker/volumes.rb
 - lib/gitlab/qa/release.rb
+- lib/gitlab/qa/report/json_test_results.rb
+- lib/gitlab/qa/report/junit_test_results.rb
 - lib/gitlab/qa/report/prepare_stage_reports.rb
 - lib/gitlab/qa/report/results_in_issues.rb
 - lib/gitlab/qa/report/summary_table.rb
+- lib/gitlab/qa/report/test_result.rb
 - lib/gitlab/qa/report/update_screenshot_path.rb
 - lib/gitlab/qa/reporter.rb
 - lib/gitlab/qa/runner.rb