[NETCONF-139] In BrokerFacade figure out and catch specific Netconf Runtime Exceptions Instead of generic RuntimeException during merge operations. Created: 03/Feb/16  Updated: 13/Aug/19

Status: Confirmed
Project: netconf
Component/s: restconf-nb
Affects Version/s: None
Fix Version/s: None

Type: Bug
Reporter: Balaji Varadaraju Assignee: Unassigned
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: All
Platform: All


External issue ID: 5197

 Description   

While fixing the issue where a Netconf device was unable to handle empty list merges, we are catching the generic RuntimeException in BrokerFacace.java since there was no clarity on all possible exceptions thrown from the underlying broker. This fixes the issue but it is better to understand the exceptions thrown by NETCONF and catch specific exceptions. Alternatively we can make the NETCONF broker to throw specific exception during merge operations and catch it.
This bug is file for the purposes of refactoring this part of the code.



 Comments   
Comment by Tomas Cere [ 03/Feb/16 ]

https://github.com/opendaylight/netconf/blob/d3ca79c031339f35df362887dde8d4f54c82cfa3/opendaylight/netconf/sal-netconf-connector/src/main/java/org/opendaylight/netconf/sal/connect/netconf/sal/tx/AbstractWriteTx.java#L135-L138

Netconf throws NetconfDocumentedException from the editConfig() but it gets caught and indeed a RuntimeException is rethrown, so a specific exception would need to be added to the interface.

Generated at Wed Feb 07 20:14:16 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.