Skip to content

[action] [PR:635] Make polling intervals in the ThermalMonitor class configurable#780

Closed
mssonicbld wants to merge 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/635
Closed

[action] [PR:635] Make polling intervals in the ThermalMonitor class configurable#780
mssonicbld wants to merge 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/635

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Default polling interval of 60s is quite high and can be unresponsive. Vendors should be able to override this polling interval

Description

Motivation and Context

How Has This Been Tested?

Signed-off-by: Sonic Build Admin sonicbld@microsoft.com

Additional Information (Optional)

Default polling interval of 60s is quite high and can be unresponsive. Vendors should be able to override this polling interval

<!-- Provide a general summary of your changes in the Title above -->

#### Description
<!--
     Describe your changes in detail
-->

#### Motivation and Context
<!--
     Why is this change required? What problem does it solve?
     If this pull request closes/resolves an open Issue, make sure you
     include the text "fixes #xxxx", "closes #xxxx" or "resolves #xxxx" here
-->

#### How Has This Been Tested?
<!--
     Please describe in detail how you tested your changes.
     Include details of your testing environment, and the tests you ran to
     see how your change affects other areas of the code, etc.
-->

Signed-off-by: Sonic Build Admin <sonicbld@microsoft.com>

#### Additional Information (Optional)
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #635

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@yejianquan yejianquan closed this Mar 25, 2026
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.

2 participants