Skip to content

Commit e1fb77b

Browse files
committed
ci/setup: Minor cleanups
- Use bash strict mode more consistently - Drop the error redirections which can mask problems as recommended by AI Signed-off-by: Colin Walters <walters@verbum.org>
1 parent 9e4cf81 commit e1fb77b

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

.github/actions/bootc-ubuntu-setup/action.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,22 +25,22 @@ runs:
2525
- name: Free up disk space on runner
2626
shell: bash
2727
run: |
28+
set -xeuo pipefail
2829
sudo df -h
2930
unwanted=('^aspnetcore-.*' '^dotnet-.*' '^llvm-.*' 'php.*' '^mongodb-.*' '^mysql-.*'
3031
azure-cli google-chrome-stable firefox mono-devel)
3132
for x in ${unwanted[@]}; do
32-
sudo apt-get remove -y $x > /dev/null
33+
sudo apt-get remove -y $x
3334
done
3435
# Start other removal operations in parallel
35-
sudo docker image prune --all --force > /dev/null &
36-
sudo rm -rf /usr/share/dotnet /opt/ghc /usr/local/lib/android &
37-
# Wait for all background processes to complete
38-
wait
36+
sudo docker image prune --all --force
37+
sudo rm -rf /usr/share/dotnet /opt/ghc /usr/local/lib/android
3938
sudo df -h
4039
# This is the default on e.g. Fedora derivatives, but not Debian
4140
- name: Enable unprivileged /dev/kvm access
4241
shell: bash
4342
run: |
43+
set -xeuo pipefail
4444
echo 'KERNEL=="kvm", GROUP="kvm", MODE="0666", OPTIONS+="static_node=kvm"' | sudo tee /etc/udev/rules.d/99-kvm4all.rules
4545
sudo udevadm control --reload-rules
4646
sudo udevadm trigger --name-match=kvm

0 commit comments

Comments
 (0)