Eli Mesika has submitted this change and it was merged.
Change subject: core: fixing dual PM handling code
......................................................................
core: fixing dual PM handling code
This patch changes the executor assignment in the run method to
this.executor as it should be.
The local parameter masks the class variable and it is not used
correctly when waitForStatus is called without this change.
this patch is mandatory for RFE
https://bugzilla.redhat.com/show_bug.cgi?id=773108
Change-Id: I0366b9615739f5e76b14beef5bc996645ee4f15b
Signed-off-by: Eli Mesika <emesika(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceVdsBaseCommand.java
1 file changed, 3 insertions(+), 3 deletions(-)
Approvals:
Eli Mesika: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/10807
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0366b9615739f5e76b14beef5bc996645ee4f15b
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>