[Bug 1546565] Re: Ownership/Permissions of vhost_user sockets for openvswitch-dpdk make them unusable by libvirt/qemu/kvm

[Michael Hudson-Doyle]

I'm patch pilot today and a little confused. Can you check my summary of
the situation?

The reason this is on the sponsorship report is to get the patch in
comment #6 uploaded to yakkety? Martin asked for it to be sent to
Debian, but as there is no dpdk in debian yet, there is no support for
it in openvswitch in debian either, so there is nowhere to send the
patch. As such the patch seems reasonable to me. Hopefully when Debian
does get support from dpdk they won't do anything gratuitously different
to what Ubuntu did...

** Changed in: openvswitch (Ubuntu)
       Status: Triaged => Incomplete

-- 
You received this bug notification because you are a member of Ubuntu
Sponsors Team, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1546565

Title:
  Ownership/Permissions of vhost_user sockets for openvswitch-dpdk make
  them unusable by libvirt/qemu/kvm

Status in dpdk package in Ubuntu:
  Fix Released
Status in openvswitch package in Ubuntu:
  Incomplete
Status in dpdk source package in Xenial:
  Fix Released
Status in openvswitch source package in Xenial:
  Triaged
Status in neutron-openvswitch package in Juju Charms Collection:
  Fix Committed

Bug description:
  As of today the vhost_user sockets created by openvswitch have root:root file ownership.
  In fact creation is actually done by code the DPDK lib, but the path is passed to it from openvswitch.

  The API called to DPDK has no notion of ownership/groups.
  It just "inherits" what the current running process has.
  But due to LP:1546556 the process ownership/group can't be changed the usual way openvsiwtch would when using dpdk.

  KVM as invoked by libvirt will run under libvirt-qemu:kvm and will
  thereby be unable to access these sockets.

  The current workaround is:
     1. wait after start of openvswitch (only then the sockets exist)
     2. chown all created vhost_iuser sockets that are to be used 
        e.g. sudo chown libvirt-qemu /var/run/openvswitch/vhost-user-1 
     3. if one wants to separate vhost_user sockets from the "rest" of openvswitch /var/run files use e.g.:
        DPDK_OPTS='[...] -vhost_sock_dir /var/run/openvswitch-vhost [...]
     X. this has to be redone every start/restart of oepnvswitch
     Y. if permissions are changed in a way that openvswitch can no more remove them on shutdown they won't re-initialize properly on the next start

  That is a severe shortcoming and not really applicable to a supported production environment.
  There are discussions ongoing about providing an option to specify owner/group/permissions of vhost_user sockets which would solve the issue.
  Unfortunately the patch series is blocked by a wider discussion about moving the dpdk configuration to the ovsdb (which makes sense, but stalls the acceptance of the patches providing the interface to modify permissions.

  Link to the last thread about moving dpdk config to ovsdb: http://comments.gmane.org/gmane.network.openvswitch.devel/59186
  Link to the last thread about making vhost_user socket user/group configurable - patch 4&5 of this: http://openvswitch.org/pipermail/dev/2015-December/063568.html
  But as mentioned it was decided to get the db config discussion done first.

  It is unsure if the patches once final will make it into openvswitch 2.5 - it would be great if they would.
  But even if not they shouldn't appear too much after and we might be able to cherry pick them?

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/dpdk/+bug/1546565/+subscriptions