diff options
| author |   Pamela Dragosh <pdragosh@research.att.com> | 2017-09-14 11:25:42 -0400 |
|---|---|---|
| committer | Pamela Dragosh <pdragosh@research.att.com> | 2017-09-14 11:29:49 -0400 |
| commit | e9b5f957f3c27c0a225d7b8fa16a220c5c98c746 (patch) | |
| tree | fd5595b25aa491ad511fb8dee019aa8a24ec7f0d /integrity-monitor | |
| parent | 65dff31a2e34b14347f1c0311420463c367afd3d (diff) | |
Logging exceptions to clear sonar critical
Just logging the exceptions anyway and remove e.printstack since
that goes to stdout/stderr.
Issue-ID: POLICY-195
Change-Id: I60616bd6ce633e2fcae266bb37eb735c6e6bb4c4
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
Diffstat (limited to 'integrity-monitor')
| -rw-r--r-- | integrity-monitor/src/main/java/org/onap/policy/common/im/IntegrityMonitor.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/integrity-monitor/src/main/java/org/onap/policy/common/im/IntegrityMonitor.java b/integrity-monitor/src/main/java/org/onap/policy/common/im/IntegrityMonitor.java index fef4b897..9d437938 100644 --- a/integrity-monitor/src/main/java/org/onap/policy/common/im/IntegrityMonitor.java +++ b/integrity-monitor/src/main/java/org/onap/policy/common/im/IntegrityMonitor.java @@ -348,7 +348,7 @@ public class IntegrityMonitor { try { new ComponentAdmin(resourceName, this, stateManager); } catch (Exception e) { - logger.error("ComponentAdmin constructor exception: " + e.toString()); + logger.error("ComponentAdmin constructor exception: " + e); } new FPManager(); @@ -906,6 +906,7 @@ public class IntegrityMonitor { //Test any subsystems that are not covered under the dependency relationship subsystemTest(); }catch (Exception e){ + logger.error(e); //This indicates a subsystemTest failure try { logger.info(resourceName + ": There has been a subsystemTest failure with error: " + e.getMessage() + " Updating this resource's state to disableDependency"); |