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 <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/17617
Reviewed-by: Michal Skrivanek <[email protected]>
Reviewed-by: Federico Simoncelli <[email protected]>
---
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 <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Martin Polednik <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: mooli tayer <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to