mirror/libbpf
1
0
Fork 0
mirror of https://github.com/netdata/libbpf.git synced 2026-04-09 01:59:06 +08:00
Code Issues Packages Projects Releases Wiki Activity

ci: don't set /dev/kvm permissions when CI user is root

Browse Source 
s390 tests are executed on selfhosted runner using root user,
avoid setting /dev/kvm permissions in such case.
This should fix CI failures like [0].
(Still necessary for x86 tests executed on standard github runners).

[0] https://github.com/libbpf/libbpf/actions/runs/6898545987/job/18768732980?pr=752

Fixes: 168630f852 ("ci: give /dev/kvm 0666 permissions inside CI runner")
Signed-off-by: Eduard Zingerman <eddyz87@gmail.com>
This commit is contained in:
Eduard Zingerman
2023-11-17 03:07:43 +02:00
committed by Andrii Nakryiko
parent 5ff0102329
commit 93f360cf4b
1 changed files with 7 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
.github/workflows/test.yml vendored
View File

@@ -37,11 +37,13 @@ jobs:
shell: bash shell: bash
run: | run: |
if [ -e /dev/kvm ]; then if [ -e /dev/kvm ]; then
echo "/dev/kvm exists, updating permissions" echo "/dev/kvm exists"
echo 'KERNEL=="kvm", GROUP="kvm", MODE="0666", OPTIONS+="static_node=kvm"' \ if [ $(id -u) != 0 ]; then
| sudo tee /etc/udev/rules.d/99-kvm4all.rules > /dev/null echo 'KERNEL=="kvm", GROUP="kvm", MODE="0666", OPTIONS+="static_node=kvm"' \
sudo udevadm control --reload-rules | sudo tee /etc/udev/rules.d/99-kvm4all.rules > /dev/null
sudo udevadm trigger --name-match=kvm sudo udevadm control --reload-rules
sudo udevadm trigger --name-match=kvm
fi
else else
echo "/dev/kvm does not exist" echo "/dev/kvm does not exist"
fi fi