On Mon, Apr 30, 2018 at 04:09:40PM +0200, Jiri Denemark wrote:
On Fri, Apr 27, 2018 at 16:25:07 +0100, Daniel P. Berrangé wrote:
> The vm name is not needed for any functional requirement, but it will be
> useful when debugging problems to identify which VM is associated with a
> filter, since UUID is not human friendly.
>
> Signed-off-by: Daniel P. Berrangé <berrange(a)redhat.com>
> ---
> src/conf/domain_nwfilter.c | 5 +++--
> src/conf/domain_nwfilter.h | 6 ++++--
> src/lxc/lxc_process.c | 2 +-
> src/nwfilter/nwfilter_driver.c | 3 ++-
> src/qemu/qemu_hotplug.c | 6 ++++--
> src/qemu/qemu_interface.c | 4 ++--
> src/qemu/qemu_process.c | 2 +-
> src/uml/uml_conf.c | 2 +-
> 8 files changed, 18 insertions(+), 12 deletions(-)
Reviewed-by: Jiri Denemark <jdenemar(a)redhat.com>
FYI, I have pushed upto & including this patch.
Regards,
Daniel
--
|:
https://berrange.com -o-
https://www.flickr.com/photos/dberrange :|
|:
https://libvirt.org -o-
https://fstop138.berrange.com :|
|:
https://entangle-photo.org -o-
https://www.instagram.com/dberrange :|