Change in ovirt-engine[master]: core : increment pool master version only if there's new mas...

[mkublin at redhat.com]

Michael Kublin has submitted this change and it was merged.

Change subject: core : increment pool master version only if there's new master
......................................................................


core : increment pool master version only if there's new master

When incrementing the pool master version, it should be incremented only
when there is a new master otherwise we might not be able to use the
pool again/perform unneeded reconstructs.

Incrementing the version always can be hazardous, an example is when we
have a pool with one domain which we deactivate, master version will be
incremented and then we won't be able to perform ConnectStoragePool (as
we will have now different master version in engine compared to vdsm),
reconstruct will also fail as there are no active domains and we won't
be able to have the pool operational again.

Change-Id: Ib292802ee2b7feeb95656d492f75dde549350c48
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/DeactivateStorageDomainCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Michael Kublin: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib292802ee2b7feeb95656d492f75dde549350c48
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>