Dan Kenigsberg has submitted this change and it was merged.
Change subject: vm: log xml from/to libvirt at 'info' level
......................................................................
vm: log xml from/to libvirt at 'info' level
The XML which VDSM sends or receive to libvirt
is a critical piece of information for troubleshooting
issues, and we must make sure to log it unless the user
really want VDSM to be really silent.
So, bump the relevant logs from 'debug' to 'info',
even if we know the xml dump is actually debug
information.
Change-Id: Ia07823512a8cdf5f05789b29bf3d4daa1a8ccb07
Signed-off-by: Francesco Romani <fromani(a)redhat.com>
Reviewed-on:
http://gerrit.ovirt.org/34358
Reviewed-by: Nir Soffer <nsoffer(a)redhat.com>
Reviewed-by: Dan Kenigsberg <danken(a)redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 22 insertions(+), 8 deletions(-)
Approvals:
Nir Soffer: Looks good to me, but someone else must approve
Dan Kenigsberg: Looks good to me, approved
Francesco Romani: Verified
--
To view, visit
http://gerrit.ovirt.org/34358
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia07823512a8cdf5f05789b29bf3d4daa1a8ccb07
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <toni(a)midokura.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server