Skip to content

Commit 3818c7a

Browse files
authored
Merge pull request #239 from openconfig/dplore/union_replace_error_string
Update error handling for union_replace CLI and OC value conflicts
2 parents 5542fd6 + 84e3ed2 commit 3818c7a

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

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)