[Tarantool-patches] [PATCH v1] gitlab-ci: save failed test results files

Alexander Turenko alexander.turenko at tarantool.org
Tue Jul 28 14:49:37 MSK 2020


> Needed for #5050

As I stated in [2] there is nothing about CI artefacts in #5050. There
is [3] about this.

> +.artifacts_reject_files_template: &artifacts_reject_files_definition
> +  artifacts:
> +    when: always
> +    paths:
> +      - "test/*/*.reject"

Are we need it when a job passes? Maybe save artefacts only for failed
jobs?

I would set `expire_in` into some large value (it is not possible to
store an artefact infinitely, gitlab docs lie), because we not always
start to investigate a flaky fail within 4 weeks. See [1] (it suggests
`expire_in: 1000 yrs`).

I also commented [2] and it seems the proposed changes will affect this
PR.

> +  variables:
> +    VARDIR: ${OSX_VARDIR}
> +  after_script:
> +    - rm -rf test/var ; mkdir test/var && cp -rfp ${OSX_VARDIR}/*.reject test/var/.
> <...>
> -		rm -rf /tmp/tnt ; \
> -		cd test && ./test-run.py --vardir /tmp/tnt --force $(TEST_RUN_EXTRA_PARAMS)
> +		rm -rf ${VARDIR} ; \
> +		cd test && ./test-run.py --vardir ${VARDIR} --force $(TEST_RUN_EXTRA_PARAMS)

I don't understand what is going on here, but I hope that we can drop it
after PR #218 (see [2]) changes.

> diff --git a/test-run b/test-run
> index d92032233..0e8da635b 160000
> --- a/test-run
> +++ b/test-run
> @@ -1 +1 @@
> -Subproject commit d92032233e10db8934ae5fed17fe3f3ae8293645
> +Subproject commit 0e8da635b76416e2ae0704c285e2bf42279f53ae

I'll update the test-run submodule separately as I always do.

[1]: https://gitlab.com/gitlab-org/gitlab/-/issues/22761#note_251722017
[2]: https://github.com/tarantool/test-run/pull/218
[3]: https://github.com/tarantool/test-run/issues/90


More information about the Tarantool-patches mailing list