diff options
author | Peter Hutterer <peter.hutterer@who-t.net> | 2020-11-23 13:19:48 +1000 |
---|---|---|
committer | Peter Hutterer <peter.hutterer@who-t.net> | 2020-11-24 07:05:58 +1000 |
commit | bbd1be7a6d043fbf451179d5570439ca29623d6f (patch) | |
tree | 142edddad4b5e095d5534a0224daa45076c0bb06 | |
parent | 60e0995360d69dd91c974cabfa85a3717ffb2e77 (diff) | |
download | libinput-bbd1be7a6d043fbf451179d5570439ca29623d6f.tar.gz |
gitlab CI: check commits/CI template generation before spinning up containers
If either of those fails, no point in trying to generate containers.
And move the MR check down to the deploy stage where it belongs.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
-rw-r--r-- | .gitlab-ci.yml | 43 | ||||
-rw-r--r-- | .gitlab-ci/ci.template | 43 |
2 files changed, 50 insertions, 36 deletions
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index f4208126..e5ee87f0 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -58,6 +58,7 @@ include: file: '/templates/ci-fairy.yml' stages: + - sanity check # CI/commit checks - prep # prep work like rebuilding the container images if there is a change - build # for actually building and testing things in a container - VM # for running the test suite in a VM @@ -132,7 +133,7 @@ variables: ################################################################# # # -# prep stage # +# sanity check stage # # # ################################################################# @@ -143,7 +144,7 @@ variables: check-ci-script: extends: - .fdo.ci-fairy - stage: prep + stage: sanity check script: - ci-fairy generate-template --verify && exit 0 || true - echo "Committed gitlab-ci.yml differs from generated gitlab-ci.yml. Please verify" @@ -156,7 +157,7 @@ check-ci-script: check-commit: extends: - .fdo.ci-fairy - stage: prep + stage: sanity check script: - ci-fairy check-commits --signed-off-by --junit-xml=results.xml except: @@ -167,21 +168,11 @@ check-commit: reports: junit: results.xml -# -# Verify that the merge request has the allow-collaboration checkbox ticked -# - -check-merge-request: - extends: - - .fdo.ci-fairy - stage: deploy - script: - - ci-fairy check-merge-request --require-allow-collaboration --junit-xml=results.xml - artifacts: - when: on_failure - reports: - junit: results.xml - allow_failure: true +################################################################# +# # +# prep stage # +# # +################################################################# # # Note: images are rebuilt weekly with a scheduled pipeline with FDO_FORCE_REBUILD set @@ -934,6 +925,22 @@ freebsd:11.2@default-build: # # ################################################################# +# +# Verify that the merge request has the allow-collaboration checkbox ticked +# + +check-merge-request: + extends: + - .fdo.ci-fairy + stage: deploy + script: + - ci-fairy check-merge-request --require-allow-collaboration --junit-xml=results.xml + artifacts: + when: on_failure + reports: + junit: results.xml + allow_failure: true + build rpm: extends: - .fdo.distribution-image@fedora diff --git a/.gitlab-ci/ci.template b/.gitlab-ci/ci.template index 68edbd67..bfe91bfb 100644 --- a/.gitlab-ci/ci.template +++ b/.gitlab-ci/ci.template @@ -48,6 +48,7 @@ include: file: '/templates/ci-fairy.yml' stages: + - sanity check # CI/commit checks - prep # prep work like rebuilding the container images if there is a change - build # for actually building and testing things in a container - VM # for running the test suite in a VM @@ -120,7 +121,7 @@ variables: ################################################################# # # -# prep stage # +# sanity check stage # # # ################################################################# @@ -131,7 +132,7 @@ variables: check-ci-script: extends: - .fdo.ci-fairy - stage: prep + stage: sanity check script: - ci-fairy generate-template --verify && exit 0 || true - echo "Committed gitlab-ci.yml differs from generated gitlab-ci.yml. Please verify" @@ -144,7 +145,7 @@ check-ci-script: check-commit: extends: - .fdo.ci-fairy - stage: prep + stage: sanity check script: - ci-fairy check-commits --signed-off-by --junit-xml=results.xml except: @@ -155,21 +156,11 @@ check-commit: reports: junit: results.xml -# -# Verify that the merge request has the allow-collaboration checkbox ticked -# - -check-merge-request: - extends: - - .fdo.ci-fairy - stage: deploy - script: - - ci-fairy check-merge-request --require-allow-collaboration --junit-xml=results.xml - artifacts: - when: on_failure - reports: - junit: results.xml - allow_failure: true +################################################################# +# # +# prep stage # +# # +################################################################# # # Note: images are rebuilt weekly with a scheduled pipeline with FDO_FORCE_REBUILD set @@ -666,6 +657,22 @@ freebsd:11.2@default-build: # # ################################################################# +# +# Verify that the merge request has the allow-collaboration checkbox ticked +# + +check-merge-request: + extends: + - .fdo.ci-fairy + stage: deploy + script: + - ci-fairy check-merge-request --require-allow-collaboration --junit-xml=results.xml + artifacts: + when: on_failure + reports: + junit: results.xml + allow_failure: true + build rpm: extends: - .fdo.distribution-image@fedora |