Omer Frenkel has submitted this change and it was merged.
Change subject: core: lock the vds manager while calling hibernate verb
......................................................................
core: lock the vds manager while calling hibernate verb
It is needed to lock the vds manager so that no statuses will be updated
while calling the hibernate verb in vdsm and setting the status of the
VM to SavingState afterwards.
The previous HibernateVDSCommand is changed to
HibernateBrokerVDSCommand, and a new HibernateVDSCommand that calls it
after the vds manager is locked, is added.
Change-Id: I2537fffccd15cb5fa30f99793c803373886c633f
Bug-Url:
https://bugzilla.redhat.com/949281
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/VDSCommandType.java
A
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/HibernateVDSCommand.java
R
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/HibernateBrokerVDSCommand.java
3 files changed, 62 insertions(+), 3 deletions(-)
Approvals:
Omer Frenkel: Looks good to me, approved
Arik Hadas: Verified
--
To view, visit
http://gerrit.ovirt.org/14520
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2537fffccd15cb5fa30f99793c803373886c633f
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>