Change in ovirt-engine[ovirt-engine-3.5]: engine: Add lock on Extend SD command

tnisan at redhat.com tnisan at redhat.com
Sun Sep 27 08:32:58 UTC 2015


Tal Nisan has submitted this change and it was merged.

Change subject: engine: Add lock on Extend SD command
......................................................................


engine: Add lock on Extend SD command

In order to prevent to run the Extend Storage Domain command twice at
the same time, an exclusive lock is added to the command.

Running this command twice will put the VG in a partial state, creating
a PV twice on the same device.

Change-Id: Ib9062ba235daa1d25298f1d295b17d15e1528c80
Backport-To: 3.6
Bug-Url: https://bugzilla.redhat.com/1265906
Signed-off-by: Fred Rolland <frolland at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ExtendSANStorageDomainCommand.java
1 file changed, 18 insertions(+), 0 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved; Passed CI tests
  Allon Mureinik: Looks good to me, but someone else must approve
  Freddy Rolland: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib9062ba235daa1d25298f1d295b17d15e1528c80
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list