From 011fbb4e78b64748c4be16c43b3deb46a5c3b23c Mon Sep 17 00:00:00 2001 From: Balaji Varadaraju Date: Wed, 14 Mar 2018 01:40:46 -0500 Subject: [PATCH] NETCONF-526 : edit-config payload default-operation needs to be set to default-value For the PUT/POST calls right now default-operation tag in the edit-config is set to "none". As per RFC with this value devices can reject a config operation when elements present in the request whose structure is not already present in the tree. For example a nested parent-child structure where parent structure does not yet exist yet in the tree may result in an error based on implementation. We should leave this field out ( which will default to merge) and let the device make the right decision. Change-Id: I8b85ef072785381897a19724dc94003b8f315bd4 Signed-off-by: Balaji Varadaraju --- .../netconf/sal/connect/netconf/sal/tx/AbstractWriteTx.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/netconf/sal-netconf-connector/src/main/java/org/opendaylight/netconf/sal/connect/netconf/sal/tx/AbstractWriteTx.java b/netconf/sal-netconf-connector/src/main/java/org/opendaylight/netconf/sal/connect/netconf/sal/tx/AbstractWriteTx.java index 7d6dd3e9e8..8c5929752c 100644 --- a/netconf/sal-netconf-connector/src/main/java/org/opendaylight/netconf/sal/connect/netconf/sal/tx/AbstractWriteTx.java +++ b/netconf/sal-netconf-connector/src/main/java/org/opendaylight/netconf/sal/connect/netconf/sal/tx/AbstractWriteTx.java @@ -106,7 +106,7 @@ public abstract class AbstractWriteTx implements DOMDataWriteTransaction { final DataContainerChild editStructure = netOps.createEditConfigStrcture(Optional.>fromNullable(data), Optional.of(ModifyAction.REPLACE), path); - editConfig(path, Optional.fromNullable(data), editStructure, Optional.of(ModifyAction.NONE), "put"); + editConfig(path, Optional.fromNullable(data), editStructure, Optional.absent(), "put"); } @Override -- 2.36.6