Skip to content

Commit 84e3ed2

Browse files
authored
Update error handling for CLI and OC value conflicts
Clarified error handling for conflicting CLI and OC values, specifying that the gRPC error string should indicate the conflicting paths.
1 parent 5542fd6 commit 84e3ed2

1 file changed

Lines changed: 7 additions & 5 deletions

File tree

rpc/gnmi/gnmi-union_replace.md

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -396,11 +396,13 @@ an acceptable alternative if option 1 cannot be supported.
396396
##### Option 1: Resolving issues with union of CLI and OC values with error
397397

398398
If a configuration item is explicitly set in CLI and OC using different values,
399-
the target MUST return an `INVALID_ARGUMENT` error. It may be necessary to set
400-
the same values in both CLI and OC to conform to constraints, such as list keys,
401-
which will need to be specified in both models if they are both targeting
402-
configuration towards the same logical list entries. It is up to the client to
403-
provide CLI and OC which are not in conflict.
399+
the target MUST return an `INVALID_ARGUMENT` error. The gRPC error string
400+
SHOULD indicate the OC path and CLI which are in conflict for human consumption.
401+
Note that it may be necessary to set the same values in both CLI and OC to
402+
conform to constraints, such as list keys, which will need to be specified in
403+
both models if they are both targeting configuration towards the same logical
404+
list entries. It is up to the client to provide CLI and OC which are not in
405+
conflict.
404406

405407
##### Option 2: Resolving issues with merging CLI and OC by overwriting
406408

0 commit comments

Comments
 (0)