From Alona Kaplan <alkaplan(a)redhat.com>:
Alona Kaplan has submitted this change and it was merged.
Change subject: engine: fix AddVdsSpmIdCommand to use correct DC id
......................................................................
engine: fix AddVdsSpmIdCommand to use correct DC id
Before the patch, when the command is run internally, the parent
transaction is suspended (by default), so the vds object, that is
loaded in getExclusiveLocks method and is cached on the command object,
is in the state that ignores the updates that are done in the parent
transaction (command). This way when the command is run from
UpdateClusterCommand and the last has updated an initially detached
cluster to belong to a DC, that new DC id isn't reflected in the vds
object that the command loads from the DB.
The patch fixes the described problem by preventing the suspension of
the parent transaction, so when the command runs internally from another
one it would see the DB data as it was updated by the parent command.
Change-Id: I2ab4e6f1332fe785ad4a19cb8e4dead3264d7815
Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsSpmIdCommand.java
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Daniel Erez: Looks good to me, approved
Yevgeny Zaspitsky: Verified
--
To view, visit
https://gerrit.ovirt.org/75822
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2ab4e6f1332fe785ad4a19cb8e4dead3264d7815
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>