From 64e9c513ce8e3f8bc83c0b7861794f9c9c9fadb3 Mon Sep 17 00:00:00 2001 From: Jamo Luhrsen Date: Fri, 30 Mar 2018 00:02:36 -0700 Subject: [PATCH] Pass error cause to service status Change-Id: I2685e124e2a6e743725340aa23c55f1a0e6c1ceb Signed-off-by: Jamo Luhrsen --- .../api/diagstatus/OpenflowPluginDiagStatusProvider.java | 5 +++++ .../openflowplugin/impl/OpenFlowPluginProviderImpl.java | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/diagstatus/OpenflowPluginDiagStatusProvider.java b/openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/diagstatus/OpenflowPluginDiagStatusProvider.java index ab8f904616..0c2ea7c9fd 100644 --- a/openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/diagstatus/OpenflowPluginDiagStatusProvider.java +++ b/openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/diagstatus/OpenflowPluginDiagStatusProvider.java @@ -47,6 +47,11 @@ public class OpenflowPluginDiagStatusProvider implements ServiceStatusProvider { }); } + public void reportStatus(ServiceState serviceState, Throwable throwable) { + LOG.debug("reporting status as {} for {}", serviceState, OPENFLOW_SERVICE_NAME); + diagStatusService.report(new ServiceDescriptor(OPENFLOW_SERVICE_NAME, throwable)); + } + public void reportStatus(ServiceState serviceState, String description) { LOG.debug("reporting status as {} for {}", serviceState, OPENFLOW_SERVICE_NAME); diagStatusService.report(new ServiceDescriptor(OPENFLOW_SERVICE_NAME, serviceState, description)); diff --git a/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/OpenFlowPluginProviderImpl.java b/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/OpenFlowPluginProviderImpl.java index d40137d98f..164f0f18cd 100644 --- a/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/OpenFlowPluginProviderImpl.java +++ b/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/OpenFlowPluginProviderImpl.java @@ -178,7 +178,7 @@ public class OpenFlowPluginProviderImpl implements @Override public void onFailure(@Nonnull final Throwable throwable) { LOG.warn("Some switchConnectionProviders failed to start.", throwable); - openflowPluginStatusMonitor.reportStatus(ServiceState.ERROR, "some switch connections failed to start"); + openflowPluginStatusMonitor.reportStatus(ServiceState.ERROR, throwable); } }, MoreExecutors.directExecutor()); } -- 2.36.6