Skip to content

Commit

Permalink
Fix the build issue when port2cable lenth define in (#5437)
Browse files Browse the repository at this point in the history
buffer_default_*.j2 because of which internal cable length never gets
define and cause failure in test case test_multinpu_cfggen.py

Signed-off-by: Abhishek Dosi <abdosi@abdosi-ubuntu-vm0.nwp1qucpfg5ejooejenqshkj3e.cx.internal.cloudapp.net>

Co-authored-by: Abhishek Dosi <abdosi@abdosi-ubuntu-vm0.nwp1qucpfg5ejooejenqshkj3e.cx.internal.cloudapp.net>
  • Loading branch information
abdosi and Abhishek Dosi committed Sep 28, 2020
1 parent 7c849ed commit 615086e
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion files/build_templates/buffers_config.j2
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ def
{%- else %}
{%- set ports2cable = {
'torrouter_server' : '5m',
'internal' : '5m',
'leafrouter_torrouter' : '40m',
'spinerouter_leafrouter' : '300m'
}
Expand All @@ -50,6 +49,9 @@ def
{%- set neighbor_role = neighbor.type %}
{%- if 'asic' == neighbor_role | lower %}
{%- set roles1 = 'internal' %}
{%- if 'internal' not in ports2cable %}
{%- set _ = ports2cable.update({'internal': '5m'}) %}
{%- endif -%}
{%- else %}
{%- set roles1 = switch_role + '_' + neighbor_role %}
{%- set roles2 = neighbor_role + '_' + switch_role %}
Expand Down

0 comments on commit 615086e

Please sign in to comment.