https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 https://jira.opendaylight.org/browse/NETCONF-248 netconf <p>Version: 1.1.0-SNAPSHOT (Boron) (master branch)</p> <p>SAL netconf connector handles invalid configuration just by logging warning message what casues that RESTCONF response indicates result code 200 OK.</p> <p>Possible solution:<br/> <a href="https://git.opendaylight.org/gerrit/gitweb?p=netconf.git;a=blob;f=netconf/sal-netconf-connector/src/main/java/org/opendaylight/netconf/sal/connect/netconf/sal/tx/WriteCandidateTx.java;h=4001c3186b385d16b376421256c9ac66c3dba224;hb=refs/heads/master#l192" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/gitweb?p=netconf.git;a=blob;f=netconf/sal-netconf-connector/src/main/java/org/opendaylight/netconf/sal/connect/netconf/sal/tx/WriteCandidateTx.java;h=4001c3186b385d16b376421256c9ac66c3dba224;hb=refs/heads/master#l192</a></p> <p>In the method onSuccess() there's a check isSuccess(). If the isSuccess returns false, then the warning message is logged. But there should be used the same handling as in the method onFailure().</p> <p>See attached logs for more details.</p> <p>Operating System: All<br/> Platform: All</p> NETCONF-248

Incorrect handling of configuration failures in SAL netconf connector

Bug Resolved Done Jakub Morvay Tomas Janciga Fri, 5 Aug 2016 08:14:22 +0000 Fri, 15 Mar 2019 22:22:28 +0000 Wed, 21 Sep 2016 12:00:53 +0000 netconf 0 6 <p>Attachment netconf_sal_connector_karaf.log has been added with description: Karaf logs showing the configuration sent, it's processing and the warning message logged instead of the appropriate handling.</p> <p>Alexis, </p> <p>If you are working on this bug, I assume it is real bug, so can you also set status to CONFIRMED?</p> <p>Done, thanks.</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/43229/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/43229/</a></p> <p>The actual solution rely on blocking get. I fixed it in: <a href="https://git.opendaylight.org/gerrit/#/c/44646/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/44646/</a><br/> where exceptions are raised in submit method.</p> <p>I abandoned posted url and I made only new patch in <a href="https://git.opendaylight.org/gerrit/#/c/43229/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/43229/</a></p> <p>Needs to be merged also to Boron</p> <p>Here's patch for stable/boron:</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/45054/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/45054/</a></p> Development External issue ID 6343 External issue URL ODL SR Target Milestone Rank 0|i01xdb: