Dan Kenigsberg has submitted this change and it was merged. Change subject: domainMonitor: Ensure that host id is released ......................................................................
domainMonitor: Ensure that host id is released Previously unexpected exception could cause the domain monitor thread to exit without releasing the host id. Stale host id will prevent removal of logical volume and the underlying devices, and prevent stopping of sanlock, requiring a reboot to upgrade a host. Now try finally block ensures that host id is is always released. Change-Id: I99b8a11b055a671d180feff92500091b3e0e1287 Signed-off-by: Nir Soffer <[email protected]> Reviewed-on: http://gerrit.ovirt.org/30922 Reviewed-by: Federico Simoncelli <[email protected]> --- M vdsm/storage/domainMonitor.py 1 file changed, 16 insertions(+), 14 deletions(-) Approvals: Nir Soffer: Verified Federico Simoncelli: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30922 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I99b8a11b055a671d180feff92500091b3e0e1287 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
