Skip to content

Commit 82873c2

Browse files
authored
[202205][Buffer] Added cable length config to buffer config template for EdgeZoneAggregator (sonic-net#14538)
Why I did it SONiC currently does not identify 'EdgeZoneAggregator' neighbor. As a result, the buffer profile attached to those interfaces uses the default cable length which could cause ingress packet drops due to insufficient headroom. Hence, there is a need to update the buffer templates to identify such neighbors and assign the same cable length as used by the T1. How I did it Modified the buffer template to identify EdgeZoneAggregator as a neighbor device type and assign it the same cable length as a T1/leaf router. How to verify it Unit tests pass, and manually checked on a 7260 to see the changes take effect.
1 parent fec2954 commit 82873c2

File tree

5 files changed

+1936
-2
lines changed

5 files changed

+1936
-2
lines changed

files/build_templates/buffers_config.j2

+3
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,9 @@ def
5353
{%- if DEVICE_NEIGHBOR_METADATA is defined and DEVICE_NEIGHBOR_METADATA[DEVICE_NEIGHBOR[local_port].name] %}
5454
{%- set neighbor = DEVICE_NEIGHBOR_METADATA[DEVICE_NEIGHBOR[local_port].name] %}
5555
{%- set neighbor_role = neighbor.type %}
56+
{%- if 'edgezoneaggregator' == neighbor_role | lower %}
57+
{%- set neighbor_role = 'LeafRouter' %}
58+
{%- endif %}
5659
{%- if 'asic' == neighbor_role | lower %}
5760
{%- set roles1 = 'internal' %}
5861
{%- if 'internal' not in ports2cable %}

0 commit comments

Comments
 (0)