Skip to content

Conversation

yuazhe
Copy link

@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:
sonic-net/sonic-platform-common#603
sonic-net/sonic-platform-daemons#690
sonic-net/sonic-utilities#4068
sonic-net/sonic-buildimage#24124
sonic-net/sonic-buildimage#24125

@mssonicbld
Copy link
Collaborator

/azp run

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

Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Collaborator

/azp run

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

Copy link

No pipelines are associated with this pull request.

@yuazhe
Copy link
Author

yuazhe commented Aug 17, 2025

LGTM, minor comments

thanks, add more details on your comment

@mssonicbld
Copy link
Collaborator

/azp run

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

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.

7 participants