* [Tarantool-patches] [PATCH v1] Correct cleanup gitlab-ci for perf jobs
@ 2020-06-27 10:16 Alexander V. Tikhonov
2020-06-29 11:42 ` Alexander Turenko
2020-07-03 11:15 ` Kirill Yukhin
0 siblings, 2 replies; 3+ messages in thread
From: Alexander V. Tikhonov @ 2020-06-27 10:16 UTC (permalink / raw)
To: Kirill Yukhin, Alexander Turenko; +Cc: tarantool-patches
Found that some perf jobs were forgot to be updated with local cleanup
routine as was done for the other jobs at commit:
892a188bc56dc7052bcbe082607a83aa73f28c5b "Correct cleanup gitlab-ci"
Follows up #5036
---
Github: https://github.com/tarantool/tarantool/tree/avtikhon/gitlab-ci-perf-cleanup
Issue: https://github.com/tarantool/tarantool/issues/5036
.gitlab-ci.yml | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 65b2fb126..6cf1aab3d 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -118,10 +118,11 @@ variables:
- ${GITLAB_MAKE} perf_run
.perf_cleanup_definition: &perf_cleanup_definition
- <<: *perf_only_definition
- stage: cleanup
- script:
- - ${GITLAB_MAKE} perf_cleanup
+ <<: *perf_only_definition
+ stage: cleanup
+ <<: *shell_cleanup_script
+ script:
+ - ${GITLAB_MAKE} perf_cleanup
# Tests
@@ -204,6 +205,7 @@ perf_bootstrap:
stage: test
tags:
- deploy
+ <<: *shell_cleanup_script
script:
- ${GITLAB_MAKE} perf_prepare
after_script:
--
2.17.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Tarantool-patches] [PATCH v1] Correct cleanup gitlab-ci for perf jobs
2020-06-27 10:16 [Tarantool-patches] [PATCH v1] Correct cleanup gitlab-ci for perf jobs Alexander V. Tikhonov
@ 2020-06-29 11:42 ` Alexander Turenko
2020-07-03 11:15 ` Kirill Yukhin
1 sibling, 0 replies; 3+ messages in thread
From: Alexander Turenko @ 2020-06-29 11:42 UTC (permalink / raw)
To: Alexander V. Tikhonov; +Cc: tarantool-patches
(I would add 'gitlab-ci:' prefix for such commits.)
It seems, we should clean the workspace before any job, because some
other job may leave a workspace dirty? Are all jobs have this action
after the commit?
Anyway, I don't mind. Feel free to ask Kirill to push it.
WBR, Alexander Turenko.
On Sat, Jun 27, 2020 at 01:16:04PM +0300, Alexander V. Tikhonov wrote:
> Found that some perf jobs were forgot to be updated with local cleanup
> routine as was done for the other jobs at commit:
>
> 892a188bc56dc7052bcbe082607a83aa73f28c5b "Correct cleanup gitlab-ci"
>
> Follows up #5036
> ---
>
> Github: https://github.com/tarantool/tarantool/tree/avtikhon/gitlab-ci-perf-cleanup
> Issue: https://github.com/tarantool/tarantool/issues/5036
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Tarantool-patches] [PATCH v1] Correct cleanup gitlab-ci for perf jobs
2020-06-27 10:16 [Tarantool-patches] [PATCH v1] Correct cleanup gitlab-ci for perf jobs Alexander V. Tikhonov
2020-06-29 11:42 ` Alexander Turenko
@ 2020-07-03 11:15 ` Kirill Yukhin
1 sibling, 0 replies; 3+ messages in thread
From: Kirill Yukhin @ 2020-07-03 11:15 UTC (permalink / raw)
To: Alexander V. Tikhonov; +Cc: tarantool-patches, Alexander Turenko
Hello,
On 27 июн 13:16, Alexander V. Tikhonov wrote:
> Found that some perf jobs were forgot to be updated with local cleanup
> routine as was done for the other jobs at commit:
>
> 892a188bc56dc7052bcbe082607a83aa73f28c5b "Correct cleanup gitlab-ci"
>
> Follows up #5036
> ---
>
> Github: https://github.com/tarantool/tarantool/tree/avtikhon/gitlab-ci-perf-cleanup
> Issue: https://github.com/tarantool/tarantool/issues/5036
I've checked your patch into 1.10, 2.3, 2.4 and master.
--
Regards, Kirill Yukhin
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-07-03 11:15 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-27 10:16 [Tarantool-patches] [PATCH v1] Correct cleanup gitlab-ci for perf jobs Alexander V. Tikhonov
2020-06-29 11:42 ` Alexander Turenko
2020-07-03 11:15 ` Kirill Yukhin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox