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