Change in ovirt-engine[master]: core : increment pool master version only if there's new mas...
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(a)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(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>