Change in ovirt-engine[master]: extmgr: use extension class loader when calling extension

yzaslavs at redhat.com yzaslavs at redhat.com
Wed May 21 11:33:19 UTC 2014


Yair Zaslavsky has submitted this change and it was merged.

Change subject: extmgr: use extension class loader when calling extension
......................................................................


extmgr: use extension class loader when calling extension

currently the context class loader is set to the jboss module that is at
top level. this does not allow the extension to use extra dependencies
it may have in its module.xml.

to solve this we set the context class loader of the extension module
before calling the extension, in this case the extension finds all its
dependencies.

Change-Id: I38031776f1d740da8f8f07f0d4a52b640534bcff
Signed-off-by: Alon Bar-Lev <alonbl at redhat.com>
---
M backend/manager/modules/extensions-api-root/extensions-api/src/main/java/org/ovirt/engine/api/extensions/Base.java
M backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionProxy.java
M backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java
3 files changed, 63 insertions(+), 49 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I38031776f1d740da8f8f07f0d4a52b640534bcff
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl 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