[Merge] ~paelzer/ubuntu/+source/nova:fix-libvirt-dependency into ~ubuntu-server-dev/ubuntu/+source/nova:master

[ChristianEhrhardt]

ChristianEhrhardt has proposed merging ~paelzer/ubuntu/+source/nova:fix-libvirt-dependency into ~ubuntu-server-dev/ubuntu/+source/nova:master.

Requested reviews:
  James Page (james-page)
Related bugs:
  Bug #1694159 in libvirt (Ubuntu): "libvirt-bin dropped conffiles but never cleaned them up on disk"
  https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/1694159

For more details, see:
https://code.launchpad.net/~paelzer/ubuntu/+source/nova/+git/nova/+merge/324778

Hi,
this is about bug 1694159.
TL;DR: get rid of old libvirt-bin dependencies.

There are also some service references to libvirt-bin in nova which are intentional (bug 1567811) for backports and such. Yet that is covered by the newer releases of libvirt having an alias of their libvirtd service to "libvirt-bin".
That said we don't want to touch these references yet (in 18.10 then where you'll no more backport to the old world it is time to remove them as we intend to drop the alias there).

But we want to replace the package dependency itself from the transitional package to the new ones - so that we can drop the transitional package at some point.

The packages were split from libvirt-bin into:
- libvirt-daemon (daemons)
- libvirt-daemon-system (run daemon as service)
- libvirt-clients (cli tools).

Currently nova-compute-libvirt depends on:
- "python-libvirt" to connect to libvirt
- libvirt-bin to have the service it can rely on

That said the correct new dependency should be libvirt-daemon-system.

There is no code reference to virsh or other cli outside some docs.
So "libvirt-clients" should not be needed as direct dependency.
-- 
Your team Ubuntu Server Developers is subscribed to branch ~ubuntu-server-dev/ubuntu/+source/nova:master.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 1163 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-reviews/attachments/20170530/b01b8bd5/attachment.diff>