[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 6/9] gitlab-ci.yml: Merge check-crypto-old jobs into the build-cry
From: |
Thomas Huth |
Subject: |
[PULL 6/9] gitlab-ci.yml: Merge check-crypto-old jobs into the build-crypto-old jobs |
Date: |
Fri, 12 Mar 2021 18:23:53 +0100 |
Both, the build-crypto-old and the check-crypto-old jobs finish reasonably
fast, and the build artifacts are only used for the single corresponding
check jobs, so there is no reason for doing the check step in a separate
job here. Thus let's stop wasting artifacts space and job scheduler over-
head by simply merging the test step into the build jobs.
Message-Id: <20210311142211.1547864-5-thuth@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
.gitlab-ci.yml | 45 ++++-----------------------------------------
1 file changed, 4 insertions(+), 41 deletions(-)
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 45303cafdd..0ada3dbb90 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -585,7 +585,7 @@ build-coroutine-sigaltstack:
#
# These jobs test old gcrypt and nettle from RHEL7
# which had some API differences.
-build-crypto-old-nettle:
+crypto-old-nettle:
<<: *native_build_job_definition
needs:
job: amd64-centos7-container
@@ -593,22 +593,9 @@ build-crypto-old-nettle:
IMAGE: centos7
TARGETS: x86_64-softmmu x86_64-linux-user
CONFIGURE_ARGS: --disable-gcrypt --enable-nettle
- MAKE_CHECK_ARGS: check-build
- artifacts:
- paths:
- - build
-
-check-crypto-old-nettle:
- <<: *native_test_job_definition
- needs:
- - job: build-crypto-old-nettle
- artifacts: true
- variables:
- IMAGE: centos7
MAKE_CHECK_ARGS: check
-
-build-crypto-old-gcrypt:
+crypto-old-gcrypt:
<<: *native_build_job_definition
needs:
job: amd64-centos7-container
@@ -616,22 +603,9 @@ build-crypto-old-gcrypt:
IMAGE: centos7
TARGETS: x86_64-softmmu x86_64-linux-user
CONFIGURE_ARGS: --disable-nettle --enable-gcrypt
- MAKE_CHECK_ARGS: check-build
- artifacts:
- paths:
- - build
-
-check-crypto-old-gcrypt:
- <<: *native_test_job_definition
- needs:
- - job: build-crypto-old-gcrypt
- artifacts: true
- variables:
- IMAGE: centos7
MAKE_CHECK_ARGS: check
-
-build-crypto-only-gnutls:
+crypto-only-gnutls:
<<: *native_build_job_definition
needs:
job: amd64-centos7-container
@@ -639,20 +613,9 @@ build-crypto-only-gnutls:
IMAGE: centos7
TARGETS: x86_64-softmmu x86_64-linux-user
CONFIGURE_ARGS: --disable-nettle --disable-gcrypt --enable-gnutls
- MAKE_CHECK_ARGS: check-build
- artifacts:
- paths:
- - build
-
-check-crypto-only-gnutls:
- <<: *native_test_job_definition
- needs:
- - job: build-crypto-only-gnutls
- artifacts: true
- variables:
- IMAGE: centos7
MAKE_CHECK_ARGS: check
+
# We don't need to exercise every backend with every front-end
build-trace-multi-user:
<<: *native_build_job_definition
--
2.27.0
- [PULL 0/9] Gitlab-CI, tests and docs, Thomas Huth, 2021/03/12
- [PULL 2/9] tests: Move benchmarks into a separate folder, Thomas Huth, 2021/03/12
- [PULL 3/9] gitlab-ci.yml: Move build-tools-and-docs-debian to a better place, Thomas Huth, 2021/03/12
- [PULL 4/9] gitlab-ci.yml: Add some missing dependencies to the jobs, Thomas Huth, 2021/03/12
- [PULL 6/9] gitlab-ci.yml: Merge check-crypto-old jobs into the build-crypto-old jobs,
Thomas Huth <=
- [PULL 7/9] tests: remove "make check-speed" in favor of "make bench", Thomas Huth, 2021/03/12
- [PULL 5/9] gitlab-ci.yml: Merge one of the coroutine jobs with the tcg-disabled job, Thomas Huth, 2021/03/12
- [PULL 1/9] tests: Move unit tests into a separate directory, Thomas Huth, 2021/03/12
- [PULL 8/9] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section, Thomas Huth, 2021/03/12
- [PULL 9/9] README: Add Documentation blurb, Thomas Huth, 2021/03/12
- Re: [PULL 0/9] Gitlab-CI, tests and docs, Peter Maydell, 2021/03/14