Hi Guido,

I'll submit the patches w/ the debconf-based warning message.

It looked good during tests. The test procedure I used is to
verify the libvirt-daemon-system package installation in the
following scenarios:

0) no libvirt-qemu user/group, no 64055 uid/gid
1) libvirt-qemu user/group with 64055 uid/gid
2) libvirt-qemu user w/ different uid
3) libvirt-qemu group w/ different gid
4) another user w/ allocated uid
5) another group w/ allocated gid

In tests 0,1 no warning message is displayed.
In tests 2-5 the warning message is displayed.

If the user chooses not to continue, the package installation
fails:

  dpkg: error processing package libvirt-daemon-system
  (--install):
   subprocess installed post-installation script returned error exit
  status 1
  Processing triggers for systemd (215-17+deb8u5) ...
  Errors were encountered while processing:
   libvirt-daemon-system

  (and the install after fixing user/group/uid/gid works correctly)

And if the user chooses to continue, the package installation
proceeds normally and finishes successfully (with different uid/gid,
of course).

I believe all that is following the expected behavior.

P.S.:

On a side note, please see [1] (thanks Christian Ehrhardt for
pointing this out.)

Do you plan to update the package with that change, and if so,
would like to see these patches following that, or change it all
after/if these patches are applied?

Thanks!

[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=621833#174

--
Mauricio Faria de Oliveira
IBM Linux Technology Center

Reply via email to