Alon Bar-Lev has uploaded a new change for review. Change subject: extmgr: cleanup: resolve log mess ......................................................................
extmgr: cleanup: resolve log mess Change-Id: Id6d92044f6e1220312287b24142e9a9ea38dad73 Signed-off-by: Alon Bar-Lev <[email protected]> --- M backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java 1 file changed, 6 insertions(+), 8 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/34/34634/1 diff --git a/backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java b/backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java index b65c8b0..90a4400 100644 --- a/backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java +++ b/backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java @@ -32,7 +32,8 @@ */ public class ExtensionsManager extends Observable { - private static final Logger logger = LoggerFactory.getLogger(ExtensionsManager.class); + private static final Logger log = LoggerFactory.getLogger(ExtensionsManager.class); + private static final Logger traceLog = LoggerFactory.getLogger(ExtensionsManager.class.getName() + ".trace"); public static final ExtKey TRACE_LOG_CONTEXT_KEY = new ExtKey("EXTENSION_MANAGER_TRACE_LOG", Logger.class, @@ -121,9 +122,6 @@ return file != null ? file.getAbsolutePath() : "N/A"; } } - - private static final Logger log = LoggerFactory.getLogger(ExtensionsManager.class.getName()); - private static final Logger traceLog = LoggerFactory.getLogger(ExtensionsManager.class.getName() + ".trace"); private static final Map<String, BindingsLoader> bindingsLoaders = new HashMap<String, BindingsLoader>() { { @@ -222,14 +220,14 @@ splitString(props.getProperty(Base.ConfigKeys.PROVIDES, "")) ); - logger.info("Loading extension '{}'", entry.name); + log.info("Loading extension '{}'", entry.name); ExtMap output = entry.extension.invoke( new ExtMap().mput( Base.InvokeKeys.COMMAND, Base.InvokeCommands.LOAD ) ); - logger.info("Extension '{}' loaded", entry.name); + log.info("Extension '{}' loaded", entry.name); entry.extension.getContext().put( TRACE_LOG_CONTEXT_KEY, @@ -307,14 +305,14 @@ extensionName)); } try { - logger.info("Initializing extension '{}'", entry.name); + log.info("Initializing extension '{}'", entry.name); ExtMap output = entry.extension.invoke( new ExtMap().mput( Base.InvokeKeys.COMMAND, Base.InvokeCommands.INITIALIZE ) ); - logger.info("Extension '{}' initialized", entry.name); + log.info("Extension '{}' initialized", entry.name); } catch (Exception ex) { log.error("Error in activating extension '{}': {}", entry.name, ex.getMessage()); if (log.isDebugEnabled()) { -- To view, visit http://gerrit.ovirt.org/34634 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Id6d92044f6e1220312287b24142e9a9ea38dad73 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
