diff mbox series

[5/9] Gitlab: Avoid downloading all artifacts in later stages

Message ID 20241208190741.4192601-6-trini@konsulko.com
State Accepted
Commit d3bb245855339426d0aaf7ef45fe108e77eb2d91
Delegated to: Tom Rini
Headers show
Series Enable arm64 host support in Gitlab | expand

Commit Message

Tom Rini Dec. 8, 2024, 5:07 p.m. UTC
In the test.py stage of the build we mark the pytest results as
artifacts to save, so that they can be used for reports. This however
leads to all of the artifacts being downloaded (and then not used) in
later stages. Optimize this out by using an empty list of dependencies
here (which is the keyword for which artifacts are needed).

Signed-off-by: Tom Rini <trini@konsulko.com>
---
 .gitlab-ci.yml | 2 ++
 1 file changed, 2 insertions(+)

Comments

Simon Glass Dec. 9, 2024, 1:29 p.m. UTC | #1
On Sun, 8 Dec 2024 at 12:08, Tom Rini <trini@konsulko.com> wrote:
>
> In the test.py stage of the build we mark the pytest results as
> artifacts to save, so that they can be used for reports. This however
> leads to all of the artifacts being downloaded (and then not used) in
> later stages. Optimize this out by using an empty list of dependencies
> here (which is the keyword for which artifacts are needed).
>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
>  .gitlab-ci.yml | 2 ++
>  1 file changed, 2 insertions(+)

Reviewed-by: Simon Glass <sjg@chromium.org>
diff mbox series

Patch

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index ae9120655b0c..696d3cb1bd74 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -111,6 +111,7 @@  stages:
 
 build all platforms in a single job:
   stage: world build
+  dependencies: []
   rules:
     - when: always
   parallel:
@@ -521,6 +522,7 @@  coreboot test.py:
     - if: $SJG_LAB != "1"
       when: manual
       allow_failure: true
+  dependencies: []
   tags: [ 'lab' ]
   script:
     - if [[ -z "${SJG_LAB}" ]]; then