FS#53980 - [libvirt] use "kvm" instead of "78" in qemu.conf?

Attached to Project: Community Packages
Opened by Tom Yan (tom.ty89) - Tuesday, 09 May 2017, 04:05 GMT
Last edited by Doug Newgard (Scimmia) - Monday, 15 May 2017, 15:01 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To No-one
Architecture All
Severity Very Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
I notice the following warning when in `systemctl status libvirtd`:

warning : virGetGroupIDByName:1072 : Group record for user '78' was not found: Invalid argument

After I ran `sed -i s/78/kvm/ /etc/libvirt/qemu.conf` and restart the libvirtd service, the warning is gone. So I guess we should set the group to "kvm" instead "78" in the PKGBUILD?


Additional info:
https://git.archlinux.org/svntogit/community.git/tree/trunk/PKGBUILD?h=packages/libvirt&id=665e70ff8e3c1afd9b011b940bd8d46e3d56e029#n87
This task depends upon

Closed by  Doug Newgard (Scimmia)
Monday, 15 May 2017, 15:01 GMT
Reason for closing:  Not a bug
Comment by Doug Newgard (Scimmia) - Wednesday, 10 May 2017, 15:38 GMT
And what else did that sed command change? The error doesn't match the complaint.
Comment by Tom Yan (tom.ty89) - Monday, 15 May 2017, 14:47 GMT
Else? What do you mean?

The warning seem to imply that libvirt only cross check the conf setting with group names but not groud ids, but we have the conf setting the group with its id instead of its name. And my sed proved that because after that the warning no longer shows up.

So I am suggesting we should do:

sed -i 's|#group =.*|group="kvm"|' src/qemu/qemu.conf

instead of:

sed -i 's|#group =.*|group="78"|' src/qemu/qemu.conf

in the PKGBUILD.

Loading...