From: "Alexander V. Tikhonov" <avtikhon@tarantool.org>
To: Kirill Yukhin <kyukhin@tarantool.org>,
Alexander Turenko <alexander.turenko@tarantool.org>
Cc: tarantool-patches@dev.tarantool.org
Subject: [Tarantool-patches] [PATCH v1] gitlab-ci: set cleanup based on docker for all
Date: Thu, 16 Jul 2020 13:11:50 +0300 [thread overview]
Message-ID: <eb09e1ec17ee1fb3c21abc2d10736ab4cfb62851.1594816569.git.avtikhon@tarantool.org> (raw)
Set cleanup based on docker for all of the jobs to avoid of fails
when the job before change could use docker and reassigned the
temporary files in the working directory. If the docker is not
available than no need to use it for cleanup and cleanup runs
using shell.
Follows up #5036
---
Github: https://github.com/tarantool/tarantool/tree/avtikhon/cleanup_gitlab-perf
Issue: https://github.com/tarantool/tarantool/issues/5036
.gitlab-ci.yml | 18 +++++++-----------
1 file changed, 7 insertions(+), 11 deletions(-)
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index ba88a26a9..1e6904eb6 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -9,13 +9,9 @@ variables:
GIT_CLEAN_COMMAND: "git clean -ffdx && git submodule foreach git clean -ffdx && git submodule foreach git status"
GIT_CLEAN_FLAGS: none
-.shell_before_script_template: &shell_cleanup_script
- before_script:
- - /bin/bash -c "${GIT_CLEAN_COMMAND}"
-
.docker_before_script_template: &docker_cleanup_script
before_script:
- - docker run -w /source -v ${PWD}:/source -i packpack/packpack:el-7 /bin/bash -c "${GIT_CLEAN_COMMAND}"
+ - /bin/bash -c "docker run -w /source -v ${PWD}:/source -i packpack/packpack:el-7 /bin/bash -c \"${GIT_CLEAN_COMMAND}\" || ( ${GIT_CLEAN_COMMAND} )"
# Jobs templates
@@ -46,14 +42,14 @@ variables:
stage: test
tags:
- docker_test
- <<: *shell_cleanup_script
+ <<: *docker_cleanup_script
.docker_test_clang8_template: &docker_test_clang8_definition
image: "${CI_REGISTRY}/${CI_PROJECT_PATH}/testing/debian-buster:latest"
stage: test
tags:
- docker_test
- <<: *shell_cleanup_script
+ <<: *docker_cleanup_script
.pack_template: &pack_definition
<<: *pack_only_definition
@@ -93,7 +89,7 @@ variables:
.osx_template: &osx_definition
stage: test
- <<: *shell_cleanup_script
+ <<: *docker_cleanup_script
script:
- ${GITLAB_MAKE} test_osx
@@ -114,14 +110,14 @@ variables:
paths:
- "*_result.txt"
- "*_t_version.txt"
- <<: *shell_cleanup_script
+ <<: *docker_cleanup_script
script:
- ${GITLAB_MAKE} perf_run
.perf_cleanup_definition: &perf_cleanup_definition
<<: *perf_only_definition
stage: cleanup
- <<: *shell_cleanup_script
+ <<: *docker_cleanup_script
script:
- ${GITLAB_MAKE} perf_cleanup
@@ -216,7 +212,7 @@ perf_bootstrap:
stage: test
tags:
- deploy
- <<: *shell_cleanup_script
+ <<: *docker_cleanup_script
script:
- ${GITLAB_MAKE} perf_prepare
after_script:
--
2.17.1
next reply other threads:[~2020-07-16 10:11 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-16 10:11 Alexander V. Tikhonov [this message]
2020-07-22 6:46 Alexander V. Tikhonov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=eb09e1ec17ee1fb3c21abc2d10736ab4cfb62851.1594816569.git.avtikhon@tarantool.org \
--to=avtikhon@tarantool.org \
--cc=alexander.turenko@tarantool.org \
--cc=kyukhin@tarantool.org \
--cc=tarantool-patches@dev.tarantool.org \
--subject='Re: [Tarantool-patches] [PATCH v1] gitlab-ci: set cleanup based on docker for all' \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox