Skip to content
This repository has been archived by the owner on Jul 31, 2024. It is now read-only.

gitlab-ci: use separate output directories for {non-,}golioth tests #413

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

mniestroj
Copy link
Collaborator

Since there were two invocations of twister in a row, first twister-out
directory was renamed during second invocation. Additionally reports/
directory contents from first run got overwritten with contents from second
run. This means that artifacts from first run were never accessible.

Run twister with --no-clean option, so that single twister-out
directory is used. Test sets from both runs are unique, which means that
the result will be a merged directory structure containing artifacts from
both runs.

Use separate reports/ directory, so that reports from both runs are
uploaded and available to be investigated.

Since there were two invocations of twister in a row, first `twister-out`
directory was renamed during second invocation. Additionally `reports/`
directory contents from first run got overwritten with contents from second
run. This means that artifacts from first run were never accessible.

Run twister with `--no-clean` option, so that single `twister-out`
directory is used. Test sets from both runs are unique, which means that
the result will be a merged directory structure containing artifacts from
both runs.

Use separate `reports/` directory, so that reports from both runs are
uploaded and available to be investigated.

Signed-off-by: Marcin Niestroj <[email protected]>
@github-actions
Copy link

Visit the preview URL for this PR (updated for commit 501d0dc):

https://golioth-zephyr-sdk-doxygen-dev--pr413-ci-separate-repo-4ye2qikf.web.app

(expires Tue, 01 Aug 2023 04:34:51 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: a389eefadf4b4b68a539327b3459dd66c142cf49

This was referenced Jul 27, 2023
@mniestroj mniestroj merged commit c0bcc6a into main Aug 1, 2023
8 checks passed
@mniestroj mniestroj deleted the ci-separate-reports branch August 1, 2023 05:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants