Change in vdsm[master]: monitor: Refresh domains only if needed

nsoffer at redhat.com nsoffer at redhat.com
Mon Feb 15 16:55:18 UTC 2016


Nir Soffer has posted comments on this change.

Change subject: monitor: Refresh domains only if needed
......................................................................


Patch Set 1:

Pavel, I don't think it make anything worse. If vdsm fail to connect to all domains before engine decide it is non-operational, we can increase the engine last check threshold (30 seconds).

Connecting to many storage domains concurrently takes time, even if we have no lvm locking. Just acquiring the sanlock host id can take up to 40 seconds.

Engine should be fixed to treat connecting to storage domain (time between storage domain is activate until it is valid and the host id is acquired) specially, this cannot use the standard logic for monitoring storage domains.

-- 
To view, visit https://gerrit.ovirt.org/53395
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I974c26fb1af3594684e00a3f9e5d64d5bfaedecd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Pavel Zhukov <pzhukov at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: No


More information about the vdsm-patches mailing list