Skip to content

Conversation

@yuazhe
Copy link
Contributor

@yuazhe yuazhe commented Sep 26, 2025

Why I did it

Following this HLD for liquid cooling sonic-net/SONiC#2032, I created this pr to add mellnox platfrom specific APIs.

This pr is part of the HLD and it has dependency on sonic-net/sonic-platform-common#603

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

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

  • 202205
  • 202211
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

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)

@yuazhe yuazhe requested a review from lguohan as a code owner September 26, 2025 06:54
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Sep 26, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: yuazhe / name: Yuanzhe (fed4843)

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yuazhe
Copy link
Contributor Author

yuazhe commented Nov 4, 2025

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 24125 in repo sonic-net/sonic-buildimage

@yuazhe
Copy link
Contributor Author

yuazhe commented Nov 4, 2025

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@yuazhe
Copy link
Contributor Author

yuazhe commented Nov 4, 2025

/azpw run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

judyjoseph
judyjoseph previously approved these changes Dec 3, 2025
@keboliu keboliu requested a review from liat-grozovik December 4, 2025 03:07
@keboliu
Copy link
Collaborator

keboliu commented Dec 4, 2025

@liat-grozovik will you be able to review and merge this PR?

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yuazhe yuazhe requested a review from judyjoseph December 5, 2025 05:28
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yuazhe
Copy link
Contributor Author

yuazhe commented Dec 8, 2025

/azpw ms_conflict

1 similar comment
@yuazhe
Copy link
Contributor Author

yuazhe commented Dec 9, 2025

/azpw ms_conflict

@yuazhe
Copy link
Contributor Author

yuazhe commented Dec 9, 2025

Hi, @liat-grozovik would you please help to merge this pr, thanks! @judyjoseph had already approved, but after update the branch this approval had been dismissed, please help to re-approve again.

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.

6 participants