Skip to content

[Arista] Fix sensors.conf for Moby#24697

Merged
yejianquan merged 1 commit intosonic-net:masterfrom
byu343:moby-sensors
Feb 23, 2026
Merged

[Arista] Fix sensors.conf for Moby#24697
yejianquan merged 1 commit intosonic-net:masterfrom
byu343:moby-sensors

Conversation

@byu343
Copy link
Copy Markdown
Contributor

@byu343 byu343 commented Dec 3, 2025

Why I did it

Add labels for sensors of Moby by sensors.conf

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

Verified that labels shown in 'sensors' output is matching the output of 'show platform temperature'

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

  • 202205
  • 202211
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

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)

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Add labels for sensors of Moby by sensors.conf

Signed-off-by: Boyang Yu <byu@arista.com>
@sdszhang sdszhang requested review from judyjoseph and r12f February 19, 2026 06:37
@sdszhang
Copy link
Copy Markdown
Contributor

Hi @judyjoseph @r12f can you help to review this one?

@r12f
Copy link
Copy Markdown
Contributor

r12f commented Feb 19, 2026

Hi @judyjoseph @r12f can you help to review this one?

Thanks Shawn! Signed off and added tags.

@yejianquan
Copy link
Copy Markdown
Contributor

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
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 .. should we run through Arista platform folks @sdszhang

@sdszhang
Copy link
Copy Markdown
Contributor

LGTM .. should we run through Arista platform folks @sdszhang

byu343 is from Arista. It's all good.

@yejianquan yejianquan merged commit cc0ab83 into sonic-net:master Feb 23, 2026
24 checks passed
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202505: #25628

FengPan-Frank pushed a commit to FengPan-Frank/sonic-buildimage that referenced this pull request Mar 6, 2026
Why I did it
Add labels for sensors of Moby by sensors.conf

Work item tracking
Microsoft ADO (number only):
How I did it
How to verify it
Verified that labels shown in 'sensors' output is matching the output of 'show platform temperature'

Add labels for sensors of Moby by sensors.conf

Signed-off-by: Boyang Yu <byu@arista.com>
Signed-off-by: Feng Pan <fenpan@microsoft.com>
dprital pushed a commit that referenced this pull request Mar 19, 2026
Why I did it
Add labels for sensors of Moby by sensors.conf

Work item tracking
Microsoft ADO (number only):
How I did it
How to verify it
Verified that labels shown in 'sensors' output is matching the output of 'show platform temperature'

Add labels for sensors of Moby by sensors.conf

Signed-off-by: Boyang Yu <byu@arista.com>
Signed-off-by: dprital <drorp@nvidia.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202511: #26276

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.

7 participants