Retrying these jobs only make the pipelines longer without any added
benefit.

Signed-off-by: Rémy Coutable <remy@rymai.me>

Add in a job to purge previous release in Review App

Add needs for all jobs which require setup-test-env and
move review-build-cng to additional stage review-prepare
to decrease wait time for jobs in pipeline

Signed-off-by: Rémy Coutable <remy@rymai.me>

Signed-off-by: Rémy Coutable <remy@rymai.me>

This also makes 'package-and-qa-manual' created for 'master' because
it's a valid use-case to run manual QA on 'master'.

Without this change, no pipeline could be created for auto-deploy
branches due to the following error:

    The form contains the following error:
    package-and-qa-manual: needs 'build-qa-image'

Signed-off-by: Rémy Coutable <remy@rymai.me>

Signed-off-by: Rémy Coutable <remy@rymai.me>

Signed-off-by: Rémy Coutable <remy@rymai.me>

Signed-off-by: Rémy Coutable <remy@rymai.me>

These branches are frequently updated and automatically deployed to real
environments, there's not really a benefit to building review apps for
them that never get used.

See https://gitlab.com/gitlab-com/gl-infra/production/issues/1023

This also has been suggested by Tomasz in the MR

Changes * to + to be more precise with acceptable branch names

Signed-off-by: Rémy Coutable <remy@rymai.me>

Signed-off-by: Rémy Coutable <remy@rymai.me>

Creates reports for parallel specs and collates them into a single
HTML report and displays results on the merge request.

Signed-off-by: Rémy Coutable <remy@rymai.me>

Signed-off-by: Rémy Coutable <remy@rymai.me>

Signed-off-by: Rémy Coutable <remy@rymai.me>

Signed-off-by: Rémy Coutable <remy@rymai.me>

Add knapsack qa report and use it to run tests in parallel

Use the RSpec runner with knapsack

The way the Knapsack runner uses exec to start rspec seems
incompatible with the way we expect it to work. Plus, it requires
specifying KNAPSACK_TEST_DIR.

Instead, we use knapsacks AllocatorBuilder to select the spec
files to run, and then start rspec as normal, via
RSpec::Core::Runner.run

This also means we can incorporate tags.

Let the job run automatically

Include KNAPSACK_TEST_FILE_PATTERN in vars

Check all defined knapsack env vars before requiring knapsack

Signed-off-by: Rémy Coutable <remy@rymai.me>

Ruby 2.6 offers better performance and memory usage:
https://www.rubyguides.com/2018/11/ruby-2-6-new-features/

Part of https://gitlab.com/gitlab-org/gitlab-ce/issues/57323

Signed-off-by: Rémy Coutable <remy@rymai.me>

By moving it to the `build` stage and adding `action: stop` to it.

Signed-off-by: Rémy Coutable <remy@rymai.me>

Also, play manual jobs once dependency jobs are done instead of polling
for the dependent jobs to be finished.

Signed-off-by: Rémy Coutable <remy@rymai.me>

Ensure that the cleanup starts right away.

Signed-off-by: Rémy Coutable <remy@rymai.me>

Signed-off-by: Rémy Coutable <remy@rymai.me>