Skip to content

Added new MLNX HWSKU ACS-MSN4410#2391

Merged
wangxin merged 3 commits intosonic-net:masterfrom
yvolynets-mlnx:add_new_hwsku
Oct 30, 2020
Merged

Added new MLNX HWSKU ACS-MSN4410#2391
wangxin merged 3 commits intosonic-net:masterfrom
yvolynets-mlnx:add_new_hwsku

Conversation

@yvolynets-mlnx
Copy link
Contributor

Signed-off-by: Yuriy Volynets yuriyv@nvidia.com

Description of PR

Summary: Updated mellanox_data.py with new HWSKU - "ACS-MSN4410"
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Approach

What is the motivation for this PR?

To extend list of supported MLNX HWSKUs.

How did you do it?

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

Signed-off-by: Yuriy Volynets <yuriyv@nvidia.com>
Copy link
Collaborator

@wangxin wangxin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure if this is complete. I noticed that for each of a Mellanox switch model, there are also other data in files like:

  • ansible/group_vars/sonic/sku-sensors-data.yml
  • ansible/group_vars/sonic/variables
  • tests/platform_tests/mellanox/minimum_table.py

@wangxin
Copy link
Collaborator

wangxin commented Oct 26, 2020

@yvolynets-mlnx For a new HWSKU, I think these files need to be updated as well, right?

  • ansible/group_vars/sonic/sku-sensors-data.yml
  • ansible/group_vars/sonic/variables
  • tests/platform_tests/mellanox/minimum_table.py

Signed-off-by: Yuriy Volynets <yuriyv@nvidia.com>
@yvolynets-mlnx
Copy link
Contributor Author

@yvolynets-mlnx For a new HWSKU, I think these files need to be updated as well, right?

* `ansible/group_vars/sonic/sku-sensors-data.yml`

* `ansible/group_vars/sonic/variables`

* `tests/platform_tests/mellanox/minimum_table.py`

Fixed. Please take a look.

@wangxin wangxin merged commit 72c0282 into sonic-net:master Oct 30, 2020
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…#12292)

linkmgrd:
* a5ac7f6 2022-10-05 | [Active-Active] Post link prober stats to state db  (sonic-net#140) (HEAD -> 202205, github/202205) [Jing Zhang]
* f4b0e53 2022-10-05 | [Active-Active] Retry config mux mode standby (sonic-net#139) [Jing Zhang]

utilities:
* a255838 2022-10-04 | [minigraph] new workflow for golden path (sonic-net#2396) (HEAD -> 202205, github/202205) [jingwenxie]
* 99425a8 2022-10-03 | [actions] Support Semgrep by Github Actions (sonic-net#2417) [Mai Bui]
* f41e4d1 2022-09-30 | Fix for show vxlan tunnel command display issue sonic-net#11902 (sonic-net#2391) [Senthil Bhava]
* e1d827e 2022-09-29 | [VxLAN]Fix Vxlan delete command to throw error when there are references (sonic-net#2404) [Sudharsan Dhamal Gopalarathnam]
* d77acf8 2022-09-28 | [doc] add documentation on automatic techsupport based on memory (sonic-net#2411) [Stepan Blyshchak]
* 2cfc75a 2022-09-28 | [doc] update "config feature" section with "--block" option (sonic-net#2409) [Stepan Blyshchak]
* 9dc8471 2022-09-28 | [Vxlanmgrd] [CPA] Update the vxlan_tunnel name len to be under IFNAMIZ to overcome netdev creation failure (sonic-net#2398) [Vivek]
* 342589e 2022-10-03 | Added cisco config platform commands (sonic-net#2242) (sonic-net#2418) [yucgu]

swss:
* 9d9f395 2022-10-04 | [intfmgr]: Enable `accept_untracked_na` kernel param (sonic-net#2436) (HEAD -> 202205, github/202205) [Lawrence Lee]
* 6b6d25d 2022-10-04 |  [orchdaemon]: Fixed sairedis record file rotation (sonic-net#2480) [Bryan Crossland]

Signed-off-by: Ying Xie <ying.xie@microsoft.com>

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
 Handle delete case for object not found (sonic-net#2391)

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
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