You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
However, it would be useful to have this for OSPFv2 as well.
I believe we could similarly include sr-igp-top in .../protocol/ospfv2/global container (openconfig-ospfv2-global.yang):
grouping ospfv2-global-structural {
description
"Top level structural grouping for OSPFv2 global parameters";
container global {
description
"Configuration and operational state parameters for settings
that are global to the OSPFv2 instance";
...
+
+ uses oc-sr:sr-igp-top;
}
}
Would such inclusion be acceptable?
Thanks.
Jardel Lucca
The text was updated successfully, but these errors were encountered:
This issue is stale because it has been open 180 days with no activity. If you wish to keep this issue active, please remove the stale label or add a comment, otherwise will be closed in 14 days.
Hi @jardel-lucca-venko, if you intend to use ospf with segment routing and are willing to create a PR with the yang model, find the implementation references confirming the support and a good reviewer with some OSPF/SR experience we would welcome the contribution!
I am not familiar with OSPF support for SR to know if your proposal is valid, but from a distance it sounds like a good approach. If you can provide sufficiently detailed implementation references/documentation I think you could demonstrate in a PR that what's being added is in fact supported by at least 2 implementations. Then the review has a good chance of going smoothly.
Hi,
We can enable SR for IS-IS with:
This
segment-routing
container is defined in openconfig-segment-routing.yang ("oc-sr")sr-igp-top
grouping.It was added in commit 55573d7 (openconfig-isis.yang):
However, it would be useful to have this for OSPFv2 as well.
I believe we could similarly include
sr-igp-top
in.../protocol/ospfv2/global
container (openconfig-ospfv2-global.yang):Would such inclusion be acceptable?
Thanks.
Jardel Lucca
The text was updated successfully, but these errors were encountered: