add hal/PolicyKit support to kvm

Bug #273764 reported by Dustin Kirkland 
6
Affects Status Importance Assigned to Milestone
kvm (Ubuntu)
Fix Released
Wishlist
Dustin Kirkland 

Bug Description

Binary package hint: kvm

The ownership of /dev/kvm is:
crw-rw---- 1 root kvm 10, 232 2008-09-23 11:04 /dev/kvm

This means that a user must be a member of kvm to use /dev/kvm.

There have been efforts elsewhere within Ubuntu Intrepid to reduce our requirements on group permissions:
 * https://wiki.ubuntu.com/DesktopTeam/Specs/Intrepid/DevicePermissions

It would be nice if we could do the same for KVM. Basically, use hal if it's available. And if not (like on most Ubuntu servers still), we fall back to group permissions.

This would help out on the desktop considerably.

:-Dustin

Related branches

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

The attached patch works for me. I request that pitti reviews and sponsors it.

Thanks,
:-Dustin

Changed in kvm:
assignee: nobody → kirkland
importance: Undecided → Wishlist
status: New → In Progress
Revision history for this message
Martin Pitt (pitti) wrote :

Nice, I didn't know about the <spawn> tag in fdi files (that's not in the FDI spec). That avoids all the addon hassle in a very nice way. The debdiff is a little redundant, why do you use install in debian/rules and additionally dh_install? Naming the files appropriately and using dh_install with "debian/foo.fdi usr/share/hal/..." etc. should be fine. I'll do that cleanup, test it myself, and get it sponsored today.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package kvm - 1:72+dfsg-1ubuntu5

---------------
kvm (1:72+dfsg-1ubuntu5) intrepid; urgency=low

  [ Dustin Kirkland ]
  * Add debian/10-kvm.fdi, debian/org.freedesktop.hal.kvm.policy: hal object
    descriptor and policy file for kvm/virt-hardware
  * debian/kvm.install; Put the hal/PolicyKit configuration in the correct
    place (LP: #273764)

  [ Martin Pitt ]
  * Simplify Dustin's dh_install handling a bit (don't use debian/rules any
    more, install straight with dh_install).

 -- Dustin Kirkland <email address hidden> Wed, 24 Sep 2008 14:00:16 +0200

Changed in kvm:
status: In Progress → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

Sponsored with this modified debdiff (Dustin, FYI).

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.