ci: make check-tree optional for branches other than main

Old branches tend to be formatted with a different versions of
clang-format, so when patches are backported, they introduce some
differences in formatting, making the check-tree job to fail.

These changes in formatting are normally small, and we don't pay much
attention to them, causing that the pipelines are always red, increasing
the work required to check if there are important failures or not.

Make check-tree optional for branches other than main. This way,
failures will be shown as a "warning", and if a pipeline only shows a
warning we don't need to inspect it.
This commit is contained in:
Íñigo Huguet
2024-12-27 09:08:43 +01:00
parent 16b932c200
commit 9e47c9bc6a
2 changed files with 13 additions and 5 deletions

View File

@@ -60,11 +60,11 @@ variables:
# #
# This is done by running `ci-fairy generate-template` and possibly bumping # This is done by running `ci-fairy generate-template` and possibly bumping
# ".default_tag". # ".default_tag".
ALPINE_TAG: 'tag-28624746b6b1' ALPINE_TAG: 'tag-77ec3d923fd6'
CENTOS_TAG: 'tag-ca7bb4b2cd94' CENTOS_TAG: 'tag-3e5d2c24e23c'
DEBIAN_TAG: 'tag-37f5e32f6021' DEBIAN_TAG: 'tag-ecad19904683'
FEDORA_TAG: 'tag-ca7bb4b2cd94' FEDORA_TAG: 'tag-3e5d2c24e23c'
UBUNTU_TAG: 'tag-37f5e32f6021' UBUNTU_TAG: 'tag-ecad19904683'
ALPINE_EXEC: 'bash .gitlab-ci/alpine-install.sh' ALPINE_EXEC: 'bash .gitlab-ci/alpine-install.sh'
CENTOS_EXEC: 'bash .gitlab-ci/fedora-install.sh' CENTOS_EXEC: 'bash .gitlab-ci/fedora-install.sh'
@@ -631,6 +631,10 @@ check-tree:
needs: needs:
- "tier1:fedora:41@prep" - "tier1:fedora:41@prep"
rules: rules:
- if: $CI_PIPELINE_SOURCE == 'merge_request_event' && $CI_MERGE_REQUEST_TARGET_BRANCH_NAME != $CI_DEFAULT_BRANCH
allow_failure: true
- if: $CI_PIPELINE_SOURCE == 'push' && $CI_COMMIT_BRANCH != $CI_DEFAULT_BRANCH
allow_failure: true
- if: $CI_PIPELINE_SOURCE != 'schedule' - if: $CI_PIPELINE_SOURCE != 'schedule'
stage: tier1 stage: tier1
script: script:

View File

@@ -214,6 +214,10 @@ check-tree:
needs: needs:
- "tier{{default_distro.tier}}:{{default_distro.name}}:{{default_distro.versions[0]}}@prep" - "tier{{default_distro.tier}}:{{default_distro.name}}:{{default_distro.versions[0]}}@prep"
rules: rules:
- if: $CI_PIPELINE_SOURCE == 'merge_request_event' && $CI_MERGE_REQUEST_TARGET_BRANCH_NAME != $CI_DEFAULT_BRANCH
allow_failure: true
- if: $CI_PIPELINE_SOURCE == 'push' && $CI_COMMIT_BRANCH != $CI_DEFAULT_BRANCH
allow_failure: true
- if: $CI_PIPELINE_SOURCE != 'schedule' - if: $CI_PIPELINE_SOURCE != 'schedule'
stage: tier1 stage: tier1
script: script: