Change in ovirt-engine[master]: core: Modify constructor to be compatible with caller method

yzaslavs at redhat.com yzaslavs at redhat.com
Fri Jun 27 02:28:39 UTC 2014


Yair Zaslavsky has submitted this change and it was merged.

Change subject: core: Modify constructor to be compatible with caller method
......................................................................


core: Modify constructor to be compatible with caller method

CreateOvfStoresForStorageDomainCommand is being called without context
although its constructor is only supported with context parameter,
so we will always get runtime exception when the command is being called.

The proposed fix remove the context from the
CreateOvfStoresForStorageDomainCommand constructor

Change-Id: Ib2cd44301ccf992541ecd0f9572fd7afde4a123d
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateOvfStoresForStorageDomainCommand.java
1 file changed, 2 insertions(+), 4 deletions(-)

Approvals:
  Yair Zaslavsky: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/29315
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib2cd44301ccf992541ecd0f9572fd7afde4a123d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list