Skip to content

Conversation

@yuazhe
Copy link
Contributor

@yuazhe yuazhe commented Jul 4, 2025

This is my HLD for the liquid cooling leakage detection, it provides a basic liquid cooling object to reflect the hardware as well as a mechanism to monitor the leakage sensors of it.

PR List:

PR title state context
[[sonic-buildimage] Add support for liquid cooling inside hardware checker GitHub issue/pull request detail GitHub pull request check contexts
[[sonic-buildimage] Add a function to get container name by asic ID] GitHub issue/pull request detail GitHub pull request check contexts
[[sonic-utilities] Add multi ASIC support for syslog rate limit feature] GitHub issue/pull request detail GitHub pull request check contexts
[[sonic-platform-common] Add new liquid cooling base object] GitHub issue/pull request detail GitHub pull request check contexts
[[sonic-platform-daemons] Add suport for liquid cooling leakage detection] GitHub issue/pull request detail GitHub pull request check contexts
[[sonic-mgmt] Add test plan and tests for liquid cooling leakage detection] GitHub issue/pull request detail GitHub pull request check contexts

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@yuazhe yuazhe requested a review from judyjoseph July 20, 2025 10:19
@volodymyrsamotiy
Copy link
Collaborator

@judyjoseph , could you please help to review?

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@bingwang-ms
Copy link
Contributor

@judyjoseph Can you help review this HLD? Thanks

Copy link
Contributor

@judyjoseph judyjoseph left a comment

Choose a reason for hiding this comment

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

LGTM, minor comments

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@yuazhe
Copy link
Contributor Author

yuazhe commented Aug 17, 2025

LGTM, minor comments

thanks, add more details on your comment

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

Copy link
Contributor

@judyjoseph judyjoseph left a comment

Choose a reason for hiding this comment

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

LGTM

@judyjoseph
Copy link
Contributor

@lguohan could you review too, anything which you would like w.r.to cooling leakage detection. Any other related working going on in cooling. thanks

@liat-grozovik liat-grozovik merged commit 28a3365 into sonic-net:master Oct 23, 2025
1 check passed
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.

8 participants