Skip to content

Remove the leaking sensor list from reset function#25112

Merged
liat-grozovik merged 2 commits intosonic-net:masterfrom
yuazhe:master_fix_lc
Jan 29, 2026
Merged

Remove the leaking sensor list from reset function#25112
liat-grozovik merged 2 commits intosonic-net:masterfrom
yuazhe:master_fix_lc

Conversation

@yuazhe
Copy link
Contributor

@yuazhe yuazhe commented Jan 16, 2026

This list is designed to record the previously found leak sensor. It can not be clear otherwise we will loose track.

Why I did it

resolve the leaking sensor recovery message can't be sent

Work item tracking
  • Microsoft ADO (number only):

How I did it

remove the leaking sensor list from reset function so we always have a track for the previous found leaking sensors, this attribute life cycle should be same as the hardware checker object.

How to verify it

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

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

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)

This list is designed to record the previously found leak sensor.
It can not be clear otherwise we will loose track.

Signed-off-by: Yuanzhe Liu <[email protected]>
@yuazhe yuazhe requested a review from lguohan as a code owner January 16, 2026 09:19
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@yuazhe yuazhe requested a review from keboliu January 16, 2026 09:20
@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).

@liat-grozovik liat-grozovik merged commit 6fb1e7c into sonic-net:master Jan 29, 2026
23 checks passed
xincunli-sonic pushed a commit to xincunli-sonic/sonic-buildimage that referenced this pull request Feb 2, 2026
This list is designed to record the previously found leak sensor. It can not be clear otherwise we will loose track.

- Why I did it
resolve the leaking sensor recovery message can't be sent

- How I did it
remove the leaking sensor list from reset function so we always have a track for the previous found leaking sensors, this attribute life cycle should be same as the hardware checker object.

Signed-off-by: Yuanzhe Liu <[email protected]>
Signed-off-by: Xincun Li <[email protected]>
FengPan-Frank pushed a commit to FengPan-Frank/sonic-buildimage that referenced this pull request Mar 6, 2026
This list is designed to record the previously found leak sensor. It can not be clear otherwise we will loose track.

- Why I did it
resolve the leaking sensor recovery message can't be sent

- How I did it
remove the leaking sensor list from reset function so we always have a track for the previous found leaking sensors, this attribute life cycle should be same as the hardware checker object.

Signed-off-by: Yuanzhe Liu <[email protected]>
Signed-off-by: Feng Pan <[email protected]>
dprital pushed a commit that referenced this pull request Mar 19, 2026
This list is designed to record the previously found leak sensor. It can not be clear otherwise we will loose track.

- Why I did it
resolve the leaking sensor recovery message can't be sent

- How I did it
remove the leaking sensor list from reset function so we always have a track for the previous found leaking sensors, this attribute life cycle should be same as the hardware checker object.

Signed-off-by: Yuanzhe Liu <[email protected]>
Signed-off-by: dprital <[email protected]>
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.

4 participants