Change in ovirt-engine[master]: core: fixing dual PM handling code

emesika at redhat.com emesika at redhat.com
Sun Jan 13 12:18:20 UTC 2013


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 at 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 at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list