Skip to content

Enable CMIS host management on Mellanox-SN4280-O28 SKU#110

Closed
vivekrnv wants to merge 1 commit intomasterfrom
sn4280_cmis_mgmt
Closed

Enable CMIS host management on Mellanox-SN4280-O28 SKU#110
vivekrnv wants to merge 1 commit intomasterfrom
sn4280_cmis_mgmt

Conversation

@vivekrnv
Copy link
Copy Markdown
Owner

Why I did it

Enable CMIS host Management feature on SN4280 platform.

How I did it

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

@vivekrnv vivekrnv requested a review from dgsudharsan August 22, 2024 16:16
@vivekrnv
Copy link
Copy Markdown
Owner Author

@noaOrMlnx Can you help with review?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

What is this file for ?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Here is the PR which explains why this file is required, https://github.com/sonic-net/sonic-buildimage/pull/19190

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Must you add this file ? can't it be linked from other SKU?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Yes, the ports are 1-28 instead of 1-32 in other platforms. Last 4 are internal ports for this platform

{
  "GLOBAL_MEDIA_SETTINGS": {
      "1-28": {
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Must you add this file ? can't it be linked from other SKU?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Same as above

@dprital
Copy link
Copy Markdown

dprital commented Aug 27, 2024

@vivekrnv , Please check that Enable / Disable CMIS Host management works well with cmis_host_mgmt.py script on this system

@vivekrnv
Copy link
Copy Markdown
Owner Author

@vivekrnv , Please check that Enable / Disable CMIS Host management works well with cmis_host_mgmt.py script on this system

You mean check if this script can enable/disable the CMIS host management properly on SN4280

@vivekrnv vivekrnv closed this Aug 27, 2024
vivekrnv pushed a commit that referenced this pull request Sep 20, 2024
…utomatically (sonic-net#20067)

#### Why I did it
src/sonic-host-services
```
* 052a797 - (HEAD -> 202311, origin/202311) Change dependency option to fix buildimage issue. (#110) (10 hours ago) [Feng-msft]
* 6ab6243 - Fix UT test data due to timestamp break. (#117) (10 hours ago) [Feng-msft]
* 7a68e40 - Update ProcessStats query by using API instead of parsing ps command. (#103) (sonic-net#154) (11 hours ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog
vivekrnv pushed a commit that referenced this pull request Sep 24, 2025
…ically (sonic-net#24025)

#### Why I did it
src/sonic-dash-ha
```
* 8f2c89c - (HEAD -> master, origin/master, origin/HEAD) Move vnet_tunnel_route_table to producer bridge (#110) (10 hours ago) [yue-fred-gao]
```
#### 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.

4 participants