openstack/kolla
Code Issues Proposed changes

Revert "nova-libvirt: fix kvm permission issue"

Browse Source 
This is not needed. If we make sure qemu use nova user.
Because nova user in group qemu.

See comment #7 of bug #1715356:
The root cause for this issue is that qemu not run with nova user on
debian.
If we set qemu running with nova user with this patch:
https://review.openstack.org/#/c/525891/
Then we need to revert commit f1b98a4925.

This reverts commit f1b98a4925.
Closes-bug: #1715356
Depends-on: I36720af0c7d3dd7c69d2404843f54c0991aea1bb

Change-Id: I62fbcf9e4ee5c3170c96576698f4ae8b66db1b74
This commit is contained in:
Xinliang Liu 2017-12-06 14:23:58 +08:00
parent 4db3c44963
commit 14cda91c4e
1 changed files with 2 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
docker/nova/nova-libvirt/extend_start.sh
View File

@ -4,18 +4,9 @@
# Fix permissions for libvirt
# Do not remove unless CentOS has been validated
if [[ -c /dev/kvm ]]; then
if [[ "${KOLLA_BASE_DISTRO}" =~ debian|ubuntu ]]; then
chmod 660 /dev/kvm
if [[ "$(uname -m)" == "aarch64" ]]; then
chown root:kvm /dev/kvm
else
chown root:qemu /dev/kvm
fi
else
chmod 666 /dev/kvm
chown root:kvm /dev/kvm
fi
fi
# Mount xenfs for libxl to work
if [[ $(lsmod | grep xenfs) ]]; then