Skip to content

Commit 1bf1140

Browse files
committed
multinode: Sanitise artifact filenames
Previously we sanitised the tmux log file name, but other files and directories can be affected, such as tempest results.
1 parent e35e4eb commit 1bf1140

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

.github/workflows/multinode.yml

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -356,13 +356,14 @@ jobs:
356356
working-directory: ${{ github.workspace }}/terraform-kayobe-multinode
357357
if: ${{ always() && steps.config_ach.outcome == 'success' }}
358358

359-
# NOTE: The tmux log rename is due to GitHub Actions not accepting files with a colon as artifacts.
360-
- name: Fix up deployment log filename
361-
run: |
362-
if [[ -f ${{ github.workspace }}/logs/tmux.kayobe:0.log ]]; then
363-
mv ${{ github.workspace }}/logs/tmux.kayobe:0.log ${{ github.workspace }}/logs/tmux.kayobe.log
364-
fi
365-
working-directory: ${{ github.workspace }}/terraform-kayobe-multinode
359+
# GitHub Actions does not accept filenames with certain characters, and
360+
# fails the upload-artifact action if any exist. The tmux log file
361+
# contains a colon, as do previous Tempest results directories.
362+
- name: Sanitise artifact filenames
363+
uses: stackhpc/stackhpc-openstack-gh-workflows/[email protected]
364+
with:
365+
path: |
366+
${{ github.workspace }}/logs/
366367
if: ${{ always() && steps.config_ach.outcome == 'success' }}
367368

368369
- name: Upload test result artifacts

0 commit comments

Comments
 (0)