summaryrefslogtreecommitdiffstats
path: root/.github
diff options
context:
space:
mode:
authorJohn Wren Kennedy <[email protected]>2021-12-01 17:18:45 -0700
committerGitHub <[email protected]>2021-12-01 16:18:45 -0800
commitddc026f59ba38ae6a33fa7b6874eaa8105ad7df3 (patch)
treedc1244764346b217f4cbebf5da2b913ba533e6e8 /.github
parent77e2756de08ccd4e8ce755f78bdcda1e3e0c55e5 (diff)
Strip colons from all test result filenames
The upload artifact functionality in github can't handle colons in filenames. The current code handles this for files under the most recent set of results. With the ability to rerun failed tests, now there can be multiple sets of results, and they all need to be processed in the same way. Reviewed-by: Brian Behlendorf <[email protected]> Reviewed-by: Tony Nguyen <[email protected]> Signed-off-by: John Kennedy <[email protected]> Closes #12815
Diffstat (limited to '.github')
-rw-r--r--.github/workflows/zfs-tests-functional.yml2
-rw-r--r--.github/workflows/zfs-tests-sanity.yml2
2 files changed, 2 insertions, 2 deletions
diff --git a/.github/workflows/zfs-tests-functional.yml b/.github/workflows/zfs-tests-functional.yml
index 46d7deace..2987cdac6 100644
--- a/.github/workflows/zfs-tests-functional.yml
+++ b/.github/workflows/zfs-tests-functional.yml
@@ -73,7 +73,7 @@ jobs:
sudo cp /var/log/syslog $RESULTS_PATH/
sudo chmod +r $RESULTS_PATH/*
# Replace ':' in dir names, actions/upload-artifact doesn't support it
- for f in $(find $RESULTS_PATH -name '*:*'); do mv "$f" "${f//:/__}"; done
+ for f in $(find /var/tmp/test_results -name '*:*'); do mv "$f" "${f//:/__}"; done
- uses: actions/upload-artifact@v2
if: failure()
with:
diff --git a/.github/workflows/zfs-tests-sanity.yml b/.github/workflows/zfs-tests-sanity.yml
index 9e0317c9d..8cba6d78d 100644
--- a/.github/workflows/zfs-tests-sanity.yml
+++ b/.github/workflows/zfs-tests-sanity.yml
@@ -69,7 +69,7 @@ jobs:
sudo cp /var/log/syslog $RESULTS_PATH/
sudo chmod +r $RESULTS_PATH/*
# Replace ':' in dir names, actions/upload-artifact doesn't support it
- for f in $(find $RESULTS_PATH -name '*:*'); do mv "$f" "${f//:/__}"; done
+ for f in $(find /var/tmp/test_results -name '*:*'); do mv "$f" "${f//:/__}"; done
- uses: actions/upload-artifact@v2
if: failure()
with: