Skip to content

Comments

[Nokia][device-data] Update Device data for Nokia chassis platform to reduce thermal low margin chatter (cheery-pick master PR 24835)#1929

Closed
mlok-nokia wants to merge 4 commits intoAzure:202405from
mlok-nokia:nokia-update-device-data
Closed

[Nokia][device-data] Update Device data for Nokia chassis platform to reduce thermal low margin chatter (cheery-pick master PR 24835)#1929
mlok-nokia wants to merge 4 commits intoAzure:202405from
mlok-nokia:nokia-update-device-data

Conversation

@mlok-nokia
Copy link
Contributor

Why I did it

Update the device data for Nokia chassis platform to reduce thermal low margin log chatter
This PR is a cherry-pick from master branch: sonic-net/sonic-buildimage#24835

Work item tracking
  • Microsoft ADO (number only):

How I did it

Reduced thermal algorithm low margin threshold for log output

How to verify it

Verify that platform_ndk.json file shows 'thermal_low_margin_threshold' value as 1

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Tested branch (Please provide the tested image version)

Description for the changelog

[Nokia][device-data] Update Device data for Nokia chassis platform to reduce thermal low margin chatter

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@mlok-nokia mlok-nokia requested a review from lguohan as a code owner January 12, 2026 18:52
@mlok-nokia
Copy link
Contributor Author

@judyjoseph Per your suggestion, this PR is a cherry-pick from master PR sonic-net/sonic-buildimage#24835. It is to reduce thermal low margin chatter. Thanks.

@mlok-nokia mlok-nokia changed the title [Nokia][device-data] Update Device data for Nokia chassis platform to reduce thermal low margin chatter [Nokia][device-data] Update Device data for Nokia chassis platform to reduce thermal low margin chatter (cheery-pick master PR 24835) Jan 12, 2026
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

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mlok-nokia
Copy link
Contributor Author

mlok-nokia commented Jan 15, 2026

@judyjoseph and @arlakshm Similar to PR #1917. The vs build failure has nothing to do with this PR. It is introduced by PR Azure/sonic-utilities.msft#250 mistakenly added two test functions for unavailable platforms. I have created a PR Azure/sonic-utilities.msft#270 to address it. Please review it as well. Thanks

@judyjoseph
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arlakshm
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mlok-nokia mlok-nokia force-pushed the nokia-update-device-data branch 2 times, most recently from 95d1844 to a0d08f8 Compare February 12, 2026 22:16
mssonicbld and others added 4 commits February 12, 2026 17:35
…AD automatically (Azure#1924)

#### Why I did it
src/sonic-utilities
```
* 8d6f34b3 - (HEAD -> 202405, origin/202405) [Chassis][ShowTechSupport] Add show chassis modules info to techSupport (Azure#268) (5 hours ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog

Signed-off-by: Sonic Automation <sonicbld@microsoft.com>
…AD automatically (Azure#1939)

#### Why I did it
src/sonic-utilities
```
* c0811ae2 - (HEAD -> 202405, origin/202405) Merge pull request Azure#270 from mlok-nokia/fix_vs_build_issue (84 minutes ago) [judyjoseph]
* 1b4645bb - [VS][UT_issue] Fix vs build UT test issue (6 hours ago) [mlok]
```
#### How I did it
#### How to verify it
#### Description for the changelog

Signed-off-by: Sonic Automation <sonicbld@microsoft.com>
…tomatically (Azure#1932)

#### Why I did it
src/sonic-swss
```
* 499e25ae - (HEAD -> 202405, origin/202405) [action] [PR:3719] [chassis] : LC reboot causing buffer_profile related errors (Azure#193) (6 hours ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog

Signed-off-by: Sonic Automation <sonicbld@microsoft.com>
… to minimize chatter

(cherry picked from commit e20052a3644b304445d24cdac88ce6c05c3fc916)
@mlok-nokia
Copy link
Contributor Author

Close this PR. This PR has been replaced by PR #1988.

@mlok-nokia mlok-nokia closed this Feb 13, 2026
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.

5 participants