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] enable host interface control by orchagent #18592

Merged
merged 1 commit into from
Apr 9, 2024

Conversation

DavidZagury
Copy link
Contributor

@DavidZagury DavidZagury commented Apr 7, 2024

Why I did it

To move the control of the netdevice’s operational status from SDK to SAI.
SAI will be responsible for updating the operational status on moving the netdev’s admin status from DOWN to UP.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Configure the SAI profile with SAI_HOSTIF_OPER_STATUS_UPDATE_BY_APP

How to verify it

Verify links go up and down correctly.

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

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

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)

@DavidZagury DavidZagury marked this pull request as ready for review April 9, 2024 06:23
@liat-grozovik liat-grozovik merged commit 3adf308 into sonic-net:master Apr 9, 2024
11 checks passed
@dprital
Copy link
Collaborator

dprital commented Apr 11, 2024

@yxieca , can you please cherry pick to 202311 ?

mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Apr 11, 2024
- Why I did it
To move the control of the netdevice’s operational status from SDK to SAI.
SAI will be responsible for updating the operational status on moving the netdev’s admin status from DOWN to UP.

- How I did it
Configure the SAI profile with SAI_HOSTIF_OPER_STATUS_UPDATE_BY_APP

- How to verify it
Verify links go up and down correctly.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202311: #18661

mssonicbld pushed a commit that referenced this pull request Apr 11, 2024
- Why I did it
To move the control of the netdevice’s operational status from SDK to SAI.
SAI will be responsible for updating the operational status on moving the netdev’s admin status from DOWN to UP.

- How I did it
Configure the SAI profile with SAI_HOSTIF_OPER_STATUS_UPDATE_BY_APP

- How to verify it
Verify links go up and down correctly.
stepanblyschak added a commit to stepanblyschak/sonic-buildimage that referenced this pull request May 21, 2024
yxieca pushed a commit that referenced this pull request May 31, 2024
… certain SKUs (#19066)

* Revert "[Mellanox] enable host interface control by orchagent (#18592)"

This reverts commit 3adf308.

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request May 31, 2024
… certain SKUs (sonic-net#19066)

* Revert "[Mellanox] enable host interface control by orchagent (sonic-net#18592)"

This reverts commit 3adf308.

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
mssonicbld pushed a commit that referenced this pull request May 31, 2024
… certain SKUs (#19066)

* Revert "[Mellanox] enable host interface control by orchagent (#18592)"

This reverts commit 3adf308.

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
arun1355492 pushed a commit to arun1355492/sonic-buildimage that referenced this pull request Jul 26, 2024
… certain SKUs (sonic-net#19066)

* Revert "[Mellanox] enable host interface control by orchagent (sonic-net#18592)"

This reverts commit 3adf308.

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants