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

Update openconfig-network-instance.yang #987

Merged
merged 6 commits into from
Nov 16, 2023
Merged

Conversation

sachendras
Copy link
Contributor

@sachendras sachendras commented Oct 28, 2023

Add clarifications to the Table-connections implementation regarding default route policy to avoid misinterpretation and maintain consistency between different NOS.

@sachendras sachendras requested a review from a team as a code owner October 28, 2023 23:43
@OpenConfigBot
Copy link

OpenConfigBot commented Oct 28, 2023

Major YANG version changes in commit 6a4e63e:

@sachendras
Copy link
Contributor Author

/gcbrun

@sachendras sachendras reopened this Nov 3, 2023
doc/network_instance_redistribution.md Outdated Show resolved Hide resolved
@sachendras
Copy link
Contributor Author

/gcbrun

Copy link
Member

@dplore dplore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to fix CI checks

@dplore
Copy link
Member

dplore commented Nov 3, 2023

/gcbrun

2 similar comments
@sachendras
Copy link
Contributor Author

/gcbrun

@sachendras
Copy link
Contributor Author

/gcbrun

@dplore dplore merged commit adf582e into openconfig:master Nov 16, 2023
5 checks passed
romeyod pushed a commit to romeyod/aftNextHop that referenced this pull request Sep 19, 2024
* Clarify network_instance_redistribution and default route policy

---------

Co-authored-by: Darren Loher <[email protected]>
Co-authored-by: Wen Bo Li <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants