diff options
| author | John Wren Kennedy <john.kennedy@delphix.com> | 2021-12-01 17:18:45 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-12-01 16:18:45 -0800 |
| commit | ddc026f59ba38ae6a33fa7b6874eaa8105ad7df3 (patch) | |
| tree | dc1244764346b217f4cbebf5da2b913ba533e6e8 | |
| parent | 77e2756de08ccd4e8ce755f78bdcda1e3e0c55e5 (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 <behlendorf1@llnl.gov>
Reviewed-by: Tony Nguyen <tony.nguyen@delphix.com>
Signed-off-by: John Kennedy <john.kennedy@delphix.com>
Closes #12815
| -rw-r--r-- | .github/workflows/zfs-tests-functional.yml | 2 | ||||
| -rw-r--r-- | .github/workflows/zfs-tests-sanity.yml | 2 |
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: |