commit fa105b2b8019906a00693ee5cfead51ed38079e3
author Yuta HIGUCHI <y-higuchi@ak.jp.nec.com> Thu Mar 01 21:16:51 2018 -0800
committer Yuta HIGUCHI <y-higuchi@opennetworking.org> Fri Mar 02 05:45:40 2018 +0000
tree 93e2ff18ec1ad41f5f0b6c9e823510267cdc2b6e
parent e848fa9b786f54987a9f25661588f70f71ead175

Add info to error log on RESTCONF manager

Change-Id: Ibeb8535b90f93578b0be432e63728adcbf474a94

apps/restconf/restconfmgr/src/main/java/org/onosproject/restconf/restconfmanager/RestconfManager.java

diff --git a/apps/restconf/restconfmgr/src/main/java/org/onosproject/restconf/restconfmanager/RestconfManager.java b/apps/restconf/restconfmgr/src/main/java/org/onosproject/restconf/restconfmanager/RestconfManager.java
index 3ef0aca..8c8d58f 100644
--- a/apps/restconf/restconfmgr/src/main/java/org/onosproject/restconf/restconfmanager/RestconfManager.java
+++ b/apps/restconf/restconfmgr/src/main/java/org/onosproject/restconf/restconfmanager/RestconfManager.java
@@ -28,6 +28,7 @@
 import org.onosproject.config.DynamicConfigService;
 import org.onosproject.config.FailedException;
 import org.onosproject.config.Filter;
+import org.onosproject.d.config.ResourceIds;
 import org.onosproject.restconf.api.RestconfError;
 import org.onosproject.restconf.api.RestconfException;
 import org.onosproject.restconf.api.RestconfRpcOutput;
@@ -151,13 +152,16 @@
 
         try {
             dynamicConfigService.createNode(rl.ridForDynConfig(), dataNode);
-        } catch (FailedException e) {
+        } catch (Exception e) {
             if (e.getMessage().startsWith("Requested node already present")) {
                 throw new RestconfException("Already exists", e,
                         RestconfError.ErrorTag.DATA_EXISTS, CONFLICT,
                         Optional.of(uri.getPath()));
             } else {
-                log.error("ERROR: DynamicConfigService: ", e);
+                log.error("ERROR: DynamicConfigService: creating {} with {}",
+                          ResourceIds.toInstanceIdentifier(rl.ridForDynConfig()),
+                          dataNode,
+                          e);
                 throw new RestconfException("ERROR: DynamicConfigService", e,
                     RestconfError.ErrorTag.OPERATION_FAILED, INTERNAL_SERVER_ERROR,
                     Optional.of(uri.getPath()));