From a34c17027be6e602fbea704b388998f53b39fa89 Mon Sep 17 00:00:00 2001 From: Karol Latecki Date: Mon, 30 Mar 2020 12:09:50 +0200 Subject: [PATCH] scripts/vagrant: fix permissions for backend files Owner and group were not set properly on Ubuntu systems, as they use different names for that. Signed-off-by: Karol Latecki Change-Id: Ie7fc49516072ce4bcd506c4c39bbd26786004c49 Signed-off-by: Karol Latecki Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/1580 Reviewed-by: Darek Stojaczyk Reviewed-by: Maciej Wawryk Reviewed-by: Tomasz Zawadzki Tested-by: SPDK CI Jenkins --- scripts/vagrant/create_nvme_img.sh | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/scripts/vagrant/create_nvme_img.sh b/scripts/vagrant/create_nvme_img.sh index e1ca76ef7..35b5ee9ab 100755 --- a/scripts/vagrant/create_nvme_img.sh +++ b/scripts/vagrant/create_nvme_img.sh @@ -65,5 +65,13 @@ if [ ! "${SYSTEM}" = "FreeBSD" ]; then fi chmod 777 $nvme_disk - chown qemu:qemu $nvme_disk + if [ $WHICH_OS == "Fedora" ]; then + chown qemu:qemu $nvme_disk + elif [ $WHICH_OS == "Ubuntu" ]; then + chown libvirt-qemu:kvm $nvme_disk + else + # That's just a wild guess for now + # TODO: needs improvement for other distros + chown libvirt-qemu:kvm $nvme_disk + fi fi