Skip to content

[Mellanox] enable host interface control by orchagent#18592

Merged
liat-grozovik merged 1 commit intosonic-net:masterfrom
DavidZagury:hostif_ctrl
Apr 9, 2024
Merged

[Mellanox] enable host interface control by orchagent#18592
liat-grozovik merged 1 commit intosonic-net:masterfrom
DavidZagury:hostif_ctrl

Conversation

@DavidZagury
Copy link
Copy Markdown
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
@DavidZagury DavidZagury requested a review from lguohan as a code owner April 9, 2024 06:23
@liat-grozovik liat-grozovik merged commit 3adf308 into sonic-net:master Apr 9, 2024
@dprital
Copy link
Copy Markdown
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
Copy Markdown
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 <[email protected]>
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 <[email protected]>
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 <[email protected]>
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