Yair Zaslavsky has submitted this change and it was merged. Change subject: aaa: creating extension-manager module ...................................................................... aaa: creating extension-manager module This module will hold all required classes for loading and managing extensions. Currnently this contains the Manage, Factory, and the Configuration related classes. The patch deals with the following: a. Moving these classes to the new module and fixing imports all over the code accordingly b. Fixing the MANIFEST.MF of the ear to add dependency on the new module. c. Fixing the get-extension to compile against authentication module as well, as currently DirectoryManager code resides there and needed for custom serialization of DirectoryUser and DirectoryGroup d. Adding a module dependency for authentication - to be dependent on extension-manager Topic: AAA Change-Id: I3c7258206880fd974934672de9fb5900192cca33 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com&gt; --- M backend/manager/modules/authentication/pom.xml M backend/manager/modules/authentication/src/main/java/org/ovirt/engine/core/authentication/AuthenticationProfileFactory.java M backend/manager/modules/authentication/src/main/java/org/ovirt/engine/core/authentication/AuthenticationProfileManager.java R backend/manager/modules/authentication/src/main/java/org/ovirt/engine/core/authentication/Authenticator.java R backend/manager/modules/authentication/src/main/java/org/ovirt/engine/core/authentication/AuthenticatorFactory.java M backend/manager/modules/authentication/src/main/java/org/ovirt/engine/core/authentication/AuthenticatorManager.java R backend/manager/modules/authentication/src/main/java/org/ovirt/engine/core/authentication/DirectoryFactory.java R backend/manager/modules/authentication/src/main/java/org/ovirt/engine/core/authentication/DirectoryManager.java M backend/manager/modules/authentication/src/main/java/org/ovirt/engine/core/authentication/header/HeaderAuthenticatorFactory.java M backend/manager/modules/authentication/src/main/java/org/ovirt/engine/core/authentication/internal/InternalAuthenticatorFactory.java M backend/manager/modules/authentication/src/main/java/org/ovirt/engine/core/authentication/internal/InternalDirectoryFactory.java M backend/manager/modules/authentication/src/main/java/org/ovirt/engine/core/authentication/nop/NopAuthenticatorFactory.java M backend/manager/modules/authentication/src/main/java/org/ovirt/engine/core/authentication/nop/NopDirectoryFactory.java M backend/manager/modules/authentication/src/main/modules/org/ovirt/engine/core/authentication/main/module.xml M backend/manager/modules/bll/pom.xml M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/authentication/provisional/ProvisionalAuthenticatorFactory.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/authentication/provisional/ProvisionalDirectoryFactory.java A backend/manager/modules/extension-manager/pom.xml R backend/manager/modules/extension-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/Configuration.java R backend/manager/modules/extension-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ConfigurationException.java R backend/manager/modules/extension-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/Factory.java R backend/manager/modules/extension-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/Manager.java R backend/manager/modules/extension-manager/src/test/java/org/ovirt/engine/core/extensions/mgr/ConfigurationTest.java M backend/manager/modules/pom.xml M ear/src/main/resources/META-INF/MANIFEST.MF M frontend/webadmin/modules/gwt-extension/pom.xml M frontend/webadmin/modules/gwt-extension/src/main/java/org/ovirt/engine/core/authentication/DirectoryGroup_CustomFieldSerializer.java M frontend/webadmin/modules/gwt-extension/src/main/java/org/ovirt/engine/core/authentication/DirectoryUser_CustomFieldSerializer.java 28 files changed, 107 insertions(+), 21 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24093 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3c7258206880fd974934672de9fb5900192cca33 Gerrit-PatchSet: 17 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com&gt; Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com&gt; Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com&gt; Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com&gt; Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server