openconfig
Clarification to usage of container `.../use-multiple-paths/*bgp/link-bandwidth-ext-community/`
Change Scope
Clarification to usage of container .../use-multiple-paths/ebgp/link-bandwidth-ext-community/
No major YANG version changes in commit ad4daf28ee49959145c94080f288d839e7667ca0
Thanks for the contribution! LGTM, except there is a version conflict to be fixed.
oc-ext:openconfig-version "9.7.2";
revision "2024-07-25" {
description
"Clarification to usage of link-bandwidth-ext-community
container under BGP use-multiple-paths`.";
reference "9.7.2";
=======
oc-ext:openconfig-version "9.8.0";
revision "2024-09-06" {
description
"Restore NONE enum value for community-type.";
reference "9.8.0";
>>>>>>> master
}
Overall LGTM from an Arista EOS perspective. The only thing I noticed is that this documentation change is using the phrasing "Usage of DMZ Link-Bandwidth extended community" which I found surprising. It makes it seem like this is a second special kind of link bandwidth extended community.
AFAIK, the "DMZ" link bandwidth terminology comes from the second link bandwidth RFC draft related to regenerating an aggregate: https://datatracker.ietf.org/doc/draft-ietf-bess-ebgp-dmz/. The main link bandwidth RFC draft which introduces the behavior described in this change makes no reference to "DMZ": https://datatracker.ietf.org/doc/draft-ietf-idr-link-bandwidth/
Overall LGTM from an Arista EOS perspective. The only thing I noticed is that this documentation change is using the phrasing "Usage of DMZ Link-Bandwidth extended community" which I found surprising. It makes it seem like this is a second special kind of link bandwidth extended community.
AFAIK, the "DMZ" link bandwidth terminology comes from the second link bandwidth RFC draft related to regenerating an aggregate: https://datatracker.ietf.org/doc/draft-ietf-bess-ebgp-dmz/. The main link bandwidth RFC draft which introduces the behavior described in this change makes no reference to "DMZ": https://datatracker.ietf.org/doc/draft-ietf-idr-link-bandwidth/
Agree. removed "DMZ "
Reviewed in Apr 22, 2025 OC Operators meeting without objection. Set last call to May 6, 2025
@rszarecki due to other PR's merging in the versions will need to be updated to resolve the conflicts here.
@rszarecki due to other PR's merging in the versions will need to be updated to resolve the conflicts here.
resolved