Liron Aravot has posted comments on this change.
Change subject: hsm: prepareForShutdown - operations order
......................................................................
Patch Set 4:
(1 comment)
https://gerrit.ovirt.org/#/c/36162/4/vdsm/storage/hsm.py
File vdsm/storage/hsm.py:
Line 3452: self.log.warning("Failed to stop RepoStats thread",
Line 3453: exc_info=True)
Line 3454:
Line 3455: self.taskMng.prepareForShutdown()
Line 3456: sp.StoragePool.cleanupMasterMount()
This may never run or vdsm may be killed in the middle.
So what
do you suggest? the current situation is problematic as well...it's either to replace
the order or to pass on the tasks status update as it needs that mount.
Line 3457: except:
Line 3458: pass
Line 3459:
Line 3460: @classmethod
--
To view, visit
https://gerrit.ovirt.org/36162
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9edd84317b08a17db80e265053edaf69582c2a51
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes