Skip to content

Update error handling for union_replace CLI and OC value conflicts #239

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
dplore wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update error handling for union_replace CLI and OC value conflicts #239

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 7 additions & 5 deletions rpc/gnmi/gnmi-union_replace.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -396,11 +396,13 @@ an acceptable alternative if option 1 cannot be supported.
##### Option 1: Resolving issues with union of CLI and OC values with error

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

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

Expand Down
Loading