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&gt; --- 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&gt; Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com&gt; Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com&gt; Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com&gt; Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com&gt; Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com&gt; Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com&gt;