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

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

Rémy Coutable authored 2 years ago and Lin Jen-Shin committed 2 years ago

Also, run against Ruby 3 in 2-hourly scheduled pipelines.

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

According to https://docs.gitlab.com/ee/install/requirements.html,
GitLab supports Redis 6.0 and up. In production we are running Redis
6.2.6, and Omnibus will soon ship with 6.2.7, so we should test with
the latest version. With this commit, CI will be testing this
combination:

- PostgreSQL 13 + Redis 6.2
- PostgreSQL 12 + Redis 6.0
- PostgreSQL 11 + Redis 5.0

At the moment Praefect in test is used with the in-memory election
strategy. We at Gitaly like to deprecate this mode and have tests
running with the default "per_repository" election strategy.

These changes make it possible to generate a Praefect configuration that
has all the setting to use this "per_repository" strategy.

Rémy Coutable authored 2 years ago and Lin Jen-Shin committed 2 years ago

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

Among other tasks, the setup helper also generates the Gitaly config.
Part of the things we configure is the internal socket directory, which
is currently being phased out by Gitaly. Instead, Gitaly provides a new
runtime directory which is intended to host all files and directories
created by Gitaly at runtime, including internal sockets.

Convert the setup helper to use the new runtime directory.

Changelog: changed

David Dieulivol authored 3 years ago and Rémy Coutable committed 3 years ago

CI/CD - Test against Redis 6 by default for postgreSQL 12, and keep Redis 5 for other postgreSQL versions

When bumping the Debian OS version, we have to be careful that we not
only bump the image but also the cache keys. Let's make it easier to
bump the image and the cache keys by defining a `DEBIAN_VERSION` so we
only have to change one line.

In !80362 we
bumped the CI images from Debian buster to bullseye. We bumped the
cache key for everything but the QA gems, which appear to be cached in
the `setup-test-env`.

Since Debian bullseye uses libc 2.29 but buster uses 2.28, this led to
errors in the `14-7-stable-ee` branch running C extensions compiled
against the former
(https://gitlab.com/gitlab-org/gitlab/-/jobs/2091504634).

To avoid any potential conflicts between Debian bullseye and buster,
bump all the `v1` cache keys to `v2`.

Debian bullseye replaces buster as the latest stable version. We have
upgraded Cloud Native GitLab to bullseye
(gitlab-org/build/CNG!888), so
let's also upgrade CI to match.

Since upgrading system libraries will likely break C extensions, we
need to bump the cache key to ensure older cached gems get recompiled
for this platform.

We also upgrade Chrome to 97 from 89 since the older version did not
appear to install on Debian bullseye due to dependency issues. Note
that Chrome 98 causes some spec failures moment due to a bug in
Chromedriver
(https://bugs.chromium.org/p/chromedriver/issues/detail?id=3999).

Changelog: changed

Rémy Coutable authored 3 years ago and Albert committed 3 years ago

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

Gitaly now builds and bundles Git v2.33
(gitaly#3815). While all Gitaly
RPCs will now use Git v2.33 as a result, it still might be useful to
match the CI version, particularly for any client-side performance
improvements.

This is needed to ensure the `update-gitaly-binaries-cache` CI job
caches the `git` binary built with Gitaly.

Relates to #343597

- Introduce a new `.if-merge-request-labels-as-if-jh` rule
- Introduce a new `add-jh-folder` job

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

Rémy Coutable authored 3 years ago and Ash McKenzie committed 3 years ago

This should save approximately 1 minutes 40 seconds.

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

This reverts merge request !66235

Changelog: changed

Updating our supported browser versions is a major change to our assets.
In order to ensure that there are no unintended side-effects, we update
all caches.

- Add .storybook-yarn-cache alias
- Push to storybook node modules cache in ci
- Use frontend rules for storybook job

DJ Mountney authored 3 years ago and Albert committed 3 years ago

This reverts merge request !63480

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>

Go 1.16 has significant memory improvements as described in
gitaly#3567.

Rémy Coutable authored 3 years ago and Albert committed 3 years ago

The debugging tasks are added to check whether RuboCop cache works as
expected or not.

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