Change in vdsm[master]: After exception on _recoverExistingVms thread we should try ...

fsimonce at redhat.com fsimonce at redhat.com
Wed Aug 21 13:14:23 UTC 2013


Federico Simoncelli has submitted this change and it was merged.

Change subject: After exception on _recoverExistingVms thread we should try again
......................................................................


After exception on _recoverExistingVms thread we should try again

Currently we fail the process on exception and gets stuck on recovery.
This might be because libvirt exception during first connect. Anyhow, we
should continue try to recover to start serving vdsm requests.

Change-Id: I80745e04c7cce50e4cd8ce12628131b76d0659a7
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=991091
Signed-off-by: Yaniv Bronhaim <ybronhei at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/17617
Reviewed-by: Michal Skrivanek <michal.skrivanek at redhat.com>
Reviewed-by: Federico Simoncelli <fsimonce at redhat.com>
---
M vdsm/clientIF.py
1 file changed, 13 insertions(+), 6 deletions(-)

Approvals:
  Yaniv Bronhaim: Verified
  Federico Simoncelli: Looks good to me, approved
  Michal Skrivanek: Looks good to me, but someone else must approve



-- 
To view, visit http://gerrit.ovirt.org/17617
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I80745e04c7cce50e4cd8ce12628131b76d0659a7
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Martin Polednik <mpoledni at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list