Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Mellanox] Bug fix: Support dynamic buffer calculation for ACS-MSN3420 and ACS-MSN4410 #51

Closed
wants to merge 1 commit into from

Conversation

stephenxs
Copy link
Owner

Why I did it

Add missed files for dynamic buffer calculation for ACS-MSN3420 and ACS-MSN4410

Signed-off-by: Stephen Sun stephens@nvidia.com

How I did it

  1. asic_table.j2: Add mapping from platform to ASIC
  2. Add buffer_dynamic.json.j2 for ACS-MSN4410.

How to verify it

Check whether the dynamic buffer calculation daemon starts succesfully.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

A picture of a cute animal (not mandatory but encouraged)

…N4410

Signed-off-by: Stephen Sun <stephens@nvidia.com>
@stephenxs stephenxs closed this Mar 22, 2021
stephenxs pushed a commit that referenced this pull request Mar 23, 2021
…dules (sonic-net#7046)

Included commits in sonic-py-swsssdk repo
```
4e0c561 2019-11-19 | read portchannel name from LAG_NAME_MAP_TABLE in COUNTERS_DB (#51) [anilkpandey]
```

Included commits in sonic-snmpagent repo
```
02dc2ce 2021-03-12 | add mock tables for LAG_NAME_MAP_TABLE in COUNTERS_DB (sonic-net#202) [Qi Luo]
```
@stephenxs stephenxs deleted the dynamic_buffer_4410_3420 branch April 10, 2021 10:53
stephenxs pushed a commit that referenced this pull request Oct 25, 2023
…ly (sonic-net#16876)

#### Why I did it
src/sonic-ztp
```
* 739470d - (HEAD -> master, origin/master, origin/HEAD) [ZTP] 'config reload' use -f to avoid system checks (#52) (4 hours ago) [Peter Yu]
* 04cd8e8 - [ZTP] bufsize=1 not supported in binary mode (#51) (4 hours ago) [Peter Yu]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants