[NETCONF-1176] Improve failed YANG patch error output Created: 05/Oct/23  Updated: 09/Nov/23  Resolved: 09/Nov/23

Status: Resolved
Project: netconf
Component/s: restconf-nb
Affects Version/s: None
Fix Version/s: 7.0.0, 4.0.9, 6.0.5, 5.0.10

Type: Task Priority: Medium
Reporter: Ivan Hrasko Assignee: Matej Šramčík
Resolution: Done Votes: 0
Labels: pt
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Issue split
split from NETCONF-1103 Incorrect response for failed PATCH r... Resolved

 Description   

On data Patch request processing (see PatchDataTransactionUtil.patchData(…))

  • each operation is processed in a sequence, per-operation status is registered
  • if there was no per-operation error then commit() is called, commit error is registered as a global error

For case when error occurs on commit the result displays global error together with per-operation status (all success) which is confusing and self contradicting.

Expected: I there is a global error then per-operation status should be omitted (dropped) bc it always displays all successes

Scope: JsonPatchBodyWriter + XmlPatchBodyWriter, ensure behavior is covered by unit tests.


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