Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Nokia-IXR7250E][Devicedata] Update the device data for Nokia IXR7250… #18063

Merged
merged 1 commit into from
Feb 8, 2024

Conversation

snider-nokia
Copy link
Contributor

@snider-nokia snider-nokia commented Feb 7, 2024

These changes adjust Nokia IXR7250 thermal sensor logging thresholds.

Why I did it

To modify the thermal sensor logging thresholds used on LC and Supervisor.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Modified the JSON based thermal logging thresholds used to determine when to log current high sensor temperature and hottest sensor margin fluctuations.

How to verify it

Verify that syslog messages indicating current (high) temperature and margin values are only logged when these respective values fluctuate by at least 5 degrees.

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

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

Tested branch (Please provide the tested image version)

  • 202205

Description for the changelog

[Nokia-IXR7250E][Devicedata] Update device data for Nokia IXR7250 for the purpose of adjusting default thermal logging thresholds

@snider-nokia
Copy link
Contributor Author

@judyjoseph, Please review and merge...

Copy link
Collaborator

@gechiang gechiang left a comment

Choose a reason for hiding this comment

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

LGTM

@gechiang
Copy link
Collaborator

gechiang commented Feb 8, 2024

MSFT ADO: 26743353
@StormLiangMS , @yxieca , Please approve backport of this bug fix to 202205 , 202305, and 202311. This change only impact Nokia 7250 thermal sensor logging level setting to remove unnecessary syslog reporting. No other functional changes.
Thanks!

@lguohan lguohan merged commit 7f3fd13 into sonic-net:master Feb 8, 2024
19 checks passed
@gechiang gechiang added the Chassis for 202205 branch PRs needed for 202205 branch in msft repo label Feb 8, 2024
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Feb 12, 2024
…E platform (thermal logging thresholds) (sonic-net#18063)

These changes adjust Nokia IXR7250 thermal sensor logging thresholds.

Why I did it
To modify the thermal sensor logging thresholds used on LC and Supervisor.

How I did it
Modified the JSON based thermal logging thresholds used to determine when to log current high sensor temperature and hottest sensor margin fluctuations.

How to verify it
Verify that syslog messages indicating current (high) temperature and margin values are only logged when these respective values fluctuate by at least 5 degrees.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202311: #18086

mssonicbld pushed a commit that referenced this pull request Feb 15, 2024
…E platform (thermal logging thresholds) (#18063)

These changes adjust Nokia IXR7250 thermal sensor logging thresholds.

Why I did it
To modify the thermal sensor logging thresholds used on LC and Supervisor.

How I did it
Modified the JSON based thermal logging thresholds used to determine when to log current high sensor temperature and hottest sensor margin fluctuations.

How to verify it
Verify that syslog messages indicating current (high) temperature and margin values are only logged when these respective values fluctuate by at least 5 degrees.
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Feb 24, 2024
…E platform (thermal logging thresholds) (sonic-net#18063)

These changes adjust Nokia IXR7250 thermal sensor logging thresholds.

Why I did it
To modify the thermal sensor logging thresholds used on LC and Supervisor.

How I did it
Modified the JSON based thermal logging thresholds used to determine when to log current high sensor temperature and hottest sensor margin fluctuations.

How to verify it
Verify that syslog messages indicating current (high) temperature and margin values are only logged when these respective values fluctuate by at least 5 degrees.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202305: #18171

StormLiangMS pushed a commit that referenced this pull request Feb 25, 2024
…E platform (thermal logging thresholds) (#18063) (#18171)

These changes adjust Nokia IXR7250 thermal sensor logging thresholds.

Why I did it
To modify the thermal sensor logging thresholds used on LC and Supervisor.

How I did it
Modified the JSON based thermal logging thresholds used to determine when to log current high sensor temperature and hottest sensor margin fluctuations.

How to verify it
Verify that syslog messages indicating current (high) temperature and margin values are only logged when these respective values fluctuate by at least 5 degrees.

Co-authored-by: snider-nokia <76123698+snider-nokia@users.noreply.github.com>
@gechiang gechiang added the Included in Chassis for 202205 Branch Indicate PR is already in MSFT repo 202205 branch label Mar 7, 2024
sonic-otn pushed a commit to Weitang-Zheng/sonic-buildimage that referenced this pull request Mar 11, 2024
…E platform (thermal logging thresholds) (sonic-net#18063)

These changes adjust Nokia IXR7250 thermal sensor logging thresholds.

Why I did it
To modify the thermal sensor logging thresholds used on LC and Supervisor.

How I did it
Modified the JSON based thermal logging thresholds used to determine when to log current high sensor temperature and hottest sensor margin fluctuations.

How to verify it
Verify that syslog messages indicating current (high) temperature and margin values are only logged when these respective values fluctuate by at least 5 degrees.
saksarav-nokia pushed a commit to saksarav-nokia/sonic-buildimage that referenced this pull request Mar 12, 2024
…E platform (thermal logging thresholds) (sonic-net#18063)

These changes adjust Nokia IXR7250 thermal sensor logging thresholds.

Why I did it
To modify the thermal sensor logging thresholds used on LC and Supervisor.

How I did it
Modified the JSON based thermal logging thresholds used to determine when to log current high sensor temperature and hottest sensor margin fluctuations.

How to verify it
Verify that syslog messages indicating current (high) temperature and margin values are only logged when these respective values fluctuate by at least 5 degrees.
mlok-nokia pushed a commit to mlok-nokia/sonic-buildimage that referenced this pull request Jun 5, 2024
…E platform (thermal logging thresholds) (sonic-net#18063)

These changes adjust Nokia IXR7250 thermal sensor logging thresholds.

Why I did it
To modify the thermal sensor logging thresholds used on LC and Supervisor.

How I did it
Modified the JSON based thermal logging thresholds used to determine when to log current high sensor temperature and hottest sensor margin fluctuations.

How to verify it
Verify that syslog messages indicating current (high) temperature and margin values are only logged when these respective values fluctuate by at least 5 degrees.
mlok-nokia pushed a commit to mlok-nokia/sonic-buildimage that referenced this pull request Jun 5, 2024
202205 manual cherry pick PRs: [chassis][voq] update the sai_post_init soc file with interrupt ids sonic-net#18066 and [Nokia-IXR7250E][Devicedata] Update the device data for Nokia IXR7250… sonic-net#18063
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.

6 participants