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

amureini at redhat.com amureini at redhat.com
Sun Sep 13 09:51:04 UTC 2015


Allon Mureinik 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/1258632
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:
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, approved
  Freddy Rolland: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib9062ba235daa1d25298f1d295b17d15e1528c80
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Fred Rolland <frolland at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list