Change in vdsm[ovirt-3.6]: clientIF: add logs during the recovery

fromani at redhat.com fromani at redhat.com
Mon Dec 14 08:26:22 UTC 2015


Francesco Romani has submitted this change and it was merged.

Change subject: clientIF: add logs during the recovery
......................................................................


clientIF: add logs during the recovery

The recovery flow is supposed to be sporadic, but it
is nevertheless very important.
The current flow is pretty opaque, and on big setups
it may take some time, leaving the admin to guess
what's going on.

This patch adds logs during the process, to make it
obvious what is happened, what is going on and what
is left to do.

Change-Id: I31dddf0a2bc760c5ad383ff6bfee9a72adc87c4f
Signed-off-by: Francesco Romani <fromani at redhat.com>
Bug-Url: https://bugzilla.redhat.com/1285236
Related-To: https://bugzilla.redhat.com/1253043
Reviewed-on: https://gerrit.ovirt.org/43770
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Continuous-Integration: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/49140
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Milan Zamazal <mzamazal at redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Martin Polednik <mpolednik at redhat.com>
---
M vdsm/clientIF.py
1 file changed, 63 insertions(+), 28 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Verified
  Martin Polednik: Looks good to me, but someone else must approve
  Milan Zamazal: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/49140
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I31dddf0a2bc760c5ad383ff6bfee9a72adc87c4f
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <mskrivan at redhat.com>
Gerrit-Reviewer: Milan Zamazal <mzamazal at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list