Uncompressed ordered map tests for ytypes #873
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When comparing two GoStructs where a YANG `ordered-by user` list is involved, it is not possible using a single Notification to represent the diff such that the Notification can do the following: 1. update a gNMI cache correctly with the atomic notification. 2. unmarshal the notification to the `orig` to get back the `modified` GoStruct. This function achieves this functionality, sharing the underlying implementation with Diff, where a disclaimer has been added to the doc comment. Other helpers have been created as well to be shared with `TogNMINotifications`. Other changes: - `TogNMINotifications` has been changed to follow the same convention of putting the non-atomic Notification as the first message instead of the last. The reason is that current code MAY be using `notifs[0]` and I don't want to break that, and this also seems slightly better from a usability perspective.
* Simplified `UnmarshalSetRequest` to always unmarshal w/r to the root node. This allows atomic Notifications to be deleted correctly.
DanG100
approved these changes
Jun 12, 2023
…#875) * Preserve some backward-compatibilities with certain render functions. Unfortunately Hyrum's law applies to ygot. I am reverting the normalizing behaviour for internal JSON generation. * Fix test imports: we can't have exported test importing symbols from … (#876) * Fix test imports: we can't have exported test importing symbols from unexported test files * fix
* Enable ordered map generation for YANG `ordered-by user` lists These satisfy the `ygot.OrderedList` interface. This is a backwards-incompatible change, but is also a bug fix at the same time. * GoOrderedList -> GoOrderedMap * Fix unkeyed list PopulateDefault generation (#877) * Fix unkeyed list PopulateDefault generation * Change interface{} -> any
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.