-
Notifications
You must be signed in to change notification settings - Fork 147
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
Create README FOR MPLS2-2 #3460
Conversation
Pull Request Test Coverage Report for Build 11251568803Details
💛 - Coveralls |
should we abandon #3106 and just review this one? |
Yes let'sgo for this Pull Request for MPLS-2.2 |
## Config paths | ||
/network-instances/network-instance/mpls/lsps/static-lsps/static-lsp/egress/config/next-hop: | ||
/network-instances/network-instance/mpls/lsps/static-lsps/static-lsp/egress/config/incoming-label: | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
## Config paths | |
/network-instances/network-instance/mpls/lsps/static-lsps/static-lsp/egress/config/next-hop: | |
/network-instances/network-instance/mpls/lsps/static-lsps/static-lsp/egress/config/incoming-label: | |
## Config paths | |
/network-instances/network-instance/protocols/protocol/static-routes/config/prefix | |
/network-instances/network-instance/protocols/protocol/static-routes/config/index: | |
/network-instances/network-instance/protocols/protocol/static-routes/next-hops/next-hop/config/next-hop: | |
/network-instances/network-instance/mpls/lsps/static-lsps/static-lsp/egress/config/name: | |
/network-instances/network-instance/mpls/lsps/static-lsps/static-lsp/egress/config/next-hop: | |
/network-instances/network-instance/mpls/lsps/static-lsps/static-lsp/egress/config/incoming-label: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey Daren,
Those paths are not invalid OCPaths
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry about that! Note, in future you can find and correct these yourself by searching at https://openconfig.net/projects/models/paths/
Here's my suggestion though:
/network-instances/network-instance/mpls/lsps/static-lsps/static-lsp/egress/config/incoming-label
/network-instances/network-instance/mpls/lsps/static-lsps/static-lsp/egress/config/next-hop
/network-instances/network-instance/protocols/protocol/static-routes/static/config/prefix
/network-instances/network-instance/protocols/protocol/static-routes/static/next-hops/next-hop/config/next-hop
/network-instances/network-instance/protocols/protocol/static-routes/static/next-hops/next-hop/config/index
Validate static LSP functionality with BGP resolved next-hop. This test verifies that the DUT can forward MPLS traffic based on a static LSP that uses a next-hop resolved via BGP.
This is to update this PR #3106 (comment) comments