Skip to content
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.

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

Marshal ordered maps into atomic gNMI notifications for TogNMINotifications #840

Merged
merged 4 commits into from
May 24, 2023

Conversation

wenovus
Copy link
Collaborator

@wenovus wenovus commented May 24, 2023

This is done by putting all values underneath an ordered map into a single, ordered list of Update values, which gets put into a separate Notification for return.

@wenovus wenovus requested a review from DanG100 May 24, 2023 00:40
@coveralls
Copy link

coveralls commented May 24, 2023

Coverage Status

Coverage: 89.982% (-0.1%) from 90.084% when pulling 10430ea on tognminotif-orderedlist into 12c092e on master.

ygot/render.go Outdated Show resolved Hide resolved
@wenovus wenovus merged commit 9e21f82 into master May 24, 2023
@wenovus wenovus deleted the tognminotif-orderedlist branch May 24, 2023 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants