[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 7/9] gitlab-ci.yml: Avoid recompiling the sources in the test jobs
From: |
Thomas Huth |
Subject: |
[PULL 7/9] gitlab-ci.yml: Avoid recompiling the sources in the test jobs |
Date: |
Wed, 27 Jan 2021 10:54:42 +0100 |
Currently, our check-system-* jobs are recompiling the whole sources
again. This happens due to the fact that the jobs are checking out
the whole source tree and required submodules again, and only try
to use the "build" directory with the binaries and object files as an
artifact from the previous stage - which simply does not work right
anymore (with the current version of meson). Due to some changed
time stamps, meson/ninja are always trying to rebuild the whole tree.
In the long run, we could likely use "meson test --no-rebuild", but
there is still some work going on in that area to improve the user
experience. So until this has been done, simply avoid recompiling the
sources with a trick: pass NINJA=":" to the make process in the test
jobs. Also check out the submodules manually before updating the
timestamps in the build folder, so that the binaries are definitely
newer that all the source files.
This saves ca. 10 - 15 minutes of precious CI cycles in each run.
Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20210126065757.403853-1-thuth@redhat.com>
Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
.gitlab-ci.yml | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index d0d8914bff..7c0db64710 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -38,9 +38,12 @@ include:
stage: test
image: $CI_REGISTRY_IMAGE/qemu/$IMAGE:latest
script:
+ - scripts/git-submodule.sh update
+ $(grep GIT_SUBMODULES build/config-host.mak | sed
's/GIT_SUBMODULES=//')
- cd build
- find . -type f -exec touch {} +
- - make $MAKE_CHECK_ARGS
+ # Avoid recompiling by hiding ninja with NINJA=":"
+ - make NINJA=":" $MAKE_CHECK_ARGS
.acceptance_template: &acceptance_definition
cache:
--
2.27.0
- [PULL 0/9] Gitlab-CI and testing improvements, Thomas Huth, 2021/01/27
- [PULL 1/9] tests/docker: Install static libc package in CentOS 7, Thomas Huth, 2021/01/27
- [PULL 2/9] gitlab-ci: Test building linux-user targets on CentOS 7, Thomas Huth, 2021/01/27
- [PULL 3/9] gitlab-ci.yml: Avoid some submodules to speed up the CI a little bit, Thomas Huth, 2021/01/27
- [PULL 4/9] configure: Only check for audio drivers if system-mode is selected, Thomas Huth, 2021/01/27
- [PULL 5/9] meson: Do not build optional libraries by default, Thomas Huth, 2021/01/27
- [PULL 6/9] gitlab-ci.yml: Exclude some redundant targets in build-without-default-features, Thomas Huth, 2021/01/27
- [PULL 7/9] gitlab-ci.yml: Avoid recompiling the sources in the test jobs,
Thomas Huth <=
- [PULL 8/9] docs/devel: Explain how acceptance tests can be skipped, Thomas Huth, 2021/01/27
- [PULL 9/9] libqtest: Rework qtest_rsp(), Thomas Huth, 2021/01/27
- Re: [PULL 0/9] Gitlab-CI and testing improvements, Peter Maydell, 2021/01/28