From: Tom Rini <trini@konsulko.com>
Date: Thu, 12 Dec 2024 22:33:36 +0000 (-0600)
Subject: Gitlab: Remove some "rules:when: always" lines
X-Git-Tag: v2025.04-rc1~17^2~16
X-Git-Url: http://git.openpandora.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=08a915719f47cde3deaf65b0c58f24d1458002d2;p=pandora-u-boot.git

Gitlab: Remove some "rules:when: always" lines

In commit 399f739be6b2 ("CI: allow jobs to be run in merge requests") we
added "rules:when: always" to many stages of the pipeline to allow for merge
requests to trigger a run. However based on current Gitlab
documentation, we should still be triggered on merge requests without
this. Furthermore the way we have things written today we always run all
stages of the CI rather than failing out early on problems, which is not
always useful. Remove these as we should still be fine with merge
requests triggering a run.

Link: https://docs.gitlab.com/ee/ci/yaml/#rules
Signed-off-by: Tom Rini <trini@konsulko.com>
---

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 01f05a04870..f5d38eaa7ac 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -32,8 +32,6 @@ stages:
   stage: test.py
   retry: 2 # QEMU may be too slow, etc.
   needs: [ "Run binman, buildman, dtoc, Kconfig and patman testsuites" ]
-  rules:
-    - when: always
   before_script:
     # Clone uboot-test-hooks
     - git config --global --add safe.directory "${CI_PROJECT_DIR}"
@@ -113,8 +111,6 @@ stages:
 build all platforms in a single job:
   stage: world build
   dependencies: []
-  rules:
-    - when: always
   parallel:
     matrix:
       - HOST: "arm64"
@@ -133,8 +129,6 @@ build all platforms in a single job:
 
 .testsuites:
   stage: testsuites
-  rules:
-    - when: always
 
 check for new CONFIG symbols outside Kconfig:
   extends: .testsuites