diff options
author | Martin Roukala (né Peres) <martin.roukala@mupuf.org> | 2023-11-21 15:31:21 +0200 |
---|---|---|
committer | Martin Roukala (né Peres) <martin.roukala@mupuf.org> | 2023-11-21 15:54:25 +0200 |
commit | a1609d76ee8940df1839b374540c0d83c054e587 (patch) | |
tree | 51874753c49e4df0e01f38ccd21b535f0010f2e0 | |
parent | 6be6b4ca711b157d0d92b4942b19eb74e71ed04f (diff) |
Revert "ci/deqp-runner: turn paths in errors into links"
This reverts commit d173584b6e41f64ba96530869c17f4fd5cbdbb98, which
inexplicably-but-reliably broke execution of `zink-radv-navi31-valve`[1].
Rather than experimenting for days on how to fix this, let's revert the
commit as it really isn't a critical thing to have. We'll rework it as
time permits.
[1] https://gitlab.freedesktop.org/mesa/mesa/-/jobs/51870356
Signed-off-by: Martin Roukala (né Peres) <martin.roukala@mupuf.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/26314>
-rwxr-xr-x | .gitlab-ci/deqp-runner.sh | 18 |
1 files changed, 5 insertions, 13 deletions
diff --git a/.gitlab-ci/deqp-runner.sh b/.gitlab-ci/deqp-runner.sh index 163b267ba45..5b90b1ff5a3 100755 --- a/.gitlab-ci/deqp-runner.sh +++ b/.gitlab-ci/deqp-runner.sh @@ -168,10 +168,8 @@ uncollapsed_section_switch deqp "deqp: deqp-runner" cat /deqp/version-log -ARTIFACTS_URL_PREFIX="https://$CI_PROJECT_ROOT_NAMESPACE.$CI_PAGES_DOMAIN/-/$CI_PROJECT_NAME/-/jobs/$CI_JOB_ID/artifacts" - +set +e if [ -z "$DEQP_SUITE" ]; then - set +e deqp-runner \ run \ --deqp $DEQP \ @@ -184,14 +182,11 @@ if [ -z "$DEQP_SUITE" ]; then $DEQP_RUNNER_OPTIONS \ -- \ $DEQP_OPTIONS - echo $? > /tmp/deqp-exit-code - set -e else # If you change the format of the suite toml filenames or the # $GPU_VERSION-{fails,flakes,skips}.txt filenames, look through the rest # of the tree for other places that need to be kept in sync (e.g. # src/amd/ci/gitlab-ci-inc.yml) - set +e deqp-runner \ suite \ --suite $INSTALL/deqp-$DEQP_SUITE.toml \ @@ -203,13 +198,10 @@ else --fraction $((CI_NODE_TOTAL * ${DEQP_FRACTION:-1})) \ --jobs ${FDO_CI_CONCURRENT:-4} \ $DEQP_RUNNER_OPTIONS - echo $? > /tmp/deqp-exit-code - set -e -fi 2>&1 | sed \ - -e "s,\"/builds/$CI_PROJECT_PATH/\(results/[^\"]\+.log\)\",$ARTIFACTS_URL_PREFIX/\1.txt,g" \ - -e "s,\"/builds/$CI_PROJECT_PATH/\(results/[^\"]\*\)\",$ARTIFACTS_URL_PREFIX/\1,g" +fi -DEQP_EXITCODE=$(cat /tmp/deqp-exit-code) +DEQP_EXITCODE=$? +set -e set +x @@ -235,7 +227,7 @@ deqp-runner junit \ --results $RESULTS/failures.csv \ --output $RESULTS/junit.xml \ --limit 50 \ - --template "See $ARTIFACTS_URL_PREFIX/results/{{testcase}}.xml" + --template "See https://$CI_PROJECT_ROOT_NAMESPACE.pages.freedesktop.org/-/$CI_PROJECT_NAME/-/jobs/$CI_JOB_ID/artifacts/results/{{testcase}}.xml" # Report the flakes to the IRC channel for monitoring (if configured): if [ -n "$FLAKES_CHANNEL" ]; then |