Change in ovirt-engine[master]: engine: Speed up SPM re-selection process.

amureini at redhat.com amureini at redhat.com
Tue Aug 20 09:15:00 UTC 2013


Allon Mureinik has submitted this change and it was merged.

Change subject: engine: Speed up SPM re-selection process.
......................................................................


engine: Speed up SPM re-selection process.

Engine trying to send stopSPM command during SPM re-selection proccess.
HSMGetAllTasksStatuses command, which is called as a part of the stopSPM
became the SPM command so technically it can get 'No SPM' error from
VDSM if the SPM is already stopped by other thread. "Not SPM" response
should be treated as kind of success so engine will move forward and
re-select the SPM.

Change-Id: I2c191ad8d4ec0fb4f5f8ff68b84fa5189ecc7114
Bug-Url: https://bugzilla.redhat.com/986961
Signed-off-by: Sergey Gotliv <sgotliv at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SpmStopVDSCommand.java
1 file changed, 17 insertions(+), 4 deletions(-)

Approvals:
  Allon Mureinik: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/18013
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I2c191ad8d4ec0fb4f5f8ff68b84fa5189ecc7114
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list