Skip to content

Commit

Permalink
[Mellanox] Fix SN5600 buffers_defaults_objects.j2 (#21543)
Browse files Browse the repository at this point in the history
- Why I did it
To fix buffers_defaults_object.j2 issues:
1. missing comma
2. missing table name
3. use of a removed profile

- How I did it
Updated the file to add comma, table name and use an existing profile

- How to verify it
config load_minigraph on the switch with Mellanox-SN5600-C256S1 SKU
  • Loading branch information
noaOrMlnx authored Jan 29, 2025
1 parent 7dc57ad commit 38403de
Showing 1 changed file with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@
"BUFFER_PORT_INGRESS_PROFILE_LIST": {
{% for port in port_names_active.split(',') %}
"{{ port }}": {
"profile_list" : "ingress_lossless_profile"
"profile_list" : "ingress_lossy_profile"
}{% if not loop.last %},{% endif %}

{% endfor %}
Expand All @@ -114,7 +114,7 @@
{% for port in port_names_inactive.split(',') %}
"{{ port }}": {
{% if dynamic_mode is defined %}
"profile_list" : "ingress_lossless_profile"
"profile_list" : "ingress_lossy_profile"
{% else %}
"profile_list" : "ingress_lossless_zero_profile"
{% endif %}
Expand Down Expand Up @@ -147,7 +147,7 @@
{%- endmacro %}

{%- macro generate_queue_buffers(port_names_active, port_names_inactive) %}

"BUFFER_QUEUE": {
{% if dynamic_mode is not defined %}
{% for port in port_names_active.split(',') %}
"{{ port }}|0": {
Expand Down Expand Up @@ -177,6 +177,7 @@
{% endfor %}
{% if port_names_inactive|length > 0 %}
{% for port in port_names_inactive.split(',') %}
{%- if loop.first -%},{%- endif -%}
"{{ port }}|1-6": {
"profile" : "egress_lossy_zero_profile"
}{% if not loop.last %},{% endif %}
Expand Down

0 comments on commit 38403de

Please sign in to comment.