Change in ovirt-engine[master]: core: lock the vds manager while calling hibernate verb

ofrenkel at redhat.com ofrenkel at redhat.com
Mon May 20 14:03:54 UTC 2013


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 at 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 at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list