From 9bf104f9348d5e7a04268399a5d68523e886d487 Mon Sep 17 00:00:00 2001 From: David Heidelberg Date: Sat, 12 Aug 2023 18:24:44 +0200 Subject: [PATCH] ci/deqp: really remove the uncompressed results.csv file Arguments are parsed in order, so the `-c` here causes that first `--rm` won't get applied and then it gets overriden by `-o` which writes the file. Sadly zstd won't report this as a issue, but that's being reported in [1], so maybe in future versions zstd will warn about it. [1] https://github.com/facebook/zstd/issues/3719 Fixes: d110299d77c9 ("ci/deqp-runner: compress results.csv before uploading it to GitLab") Signed-off-by: David Heidelberg Part-of: --- .gitlab-ci/deqp-runner.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.gitlab-ci/deqp-runner.sh b/.gitlab-ci/deqp-runner.sh index ae79d534862..4dae99c1f5c 100755 --- a/.gitlab-ci/deqp-runner.sh +++ b/.gitlab-ci/deqp-runner.sh @@ -235,7 +235,7 @@ fi # Compress results.csv to save on bandwidth during the upload of artifacts to # GitLab. This reduces the size in a VKCTS run from 135 to 7.6MB, and takes # 0.17s on a Ryzen 5950X (16 threads, 0.95s when limited to 1 thread). -zstd --rm -T0 -8qc $RESULTS/results.csv -o $RESULTS/results.csv.zst +zstd --rm -T0 -8q "$RESULTS/results.csv" -o "$RESULTS/results.csv.zst" section_end test_post_process