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

[Mellanox] Add new sensor conf to support SN4410 A1 system #8379

Merged
merged 2 commits into from
Aug 12, 2021

Conversation

keboliu
Copy link
Collaborator

@keboliu keboliu commented Aug 9, 2021

Why I did it

New SN410 A1 system has a different sensor layout with A0 system, needs a new sensor conf file to support it.

How I did it

Since the SN4410 A1 system use exactly the same sensor layout as the SN4700 A1 system, so add a symbol link linking to the SN4700 A1 sensor conf file to reuse.

How to verify it

Run sensor test against the SN4410 A1 system;
Run platform related regression test against the SN4410 A1 system

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

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

@qiluo-msft qiluo-msft merged commit 9d2ccda into sonic-net:master Aug 12, 2021
@keboliu keboliu deleted the Ocelot-A1 branch August 12, 2021 01:10
judyjoseph pushed a commit that referenced this pull request Aug 25, 2021
#### Why I did it

New SN410 A1 system has a different sensor layout with A0 system, needs a new sensor conf file to support it.

#### How I did it

Since the SN4410 A1 system use exactly the same sensor layout as the SN4700 A1 system, so add a symbol link linking to the SN4700 A1 sensor conf file to reuse.

#### How to verify it

Run sensor test against the SN4410 A1 system;
Run platform related regression test against the SN4410 A1 system
keboliu added a commit to keboliu/sonic-buildimage that referenced this pull request Jul 25, 2022
…#8379)

#### Why I did it

New SN410 A1 system has a different sensor layout with A0 system, needs a new sensor conf file to support it.

#### How I did it

Since the SN4410 A1 system use exactly the same sensor layout as the SN4700 A1 system, so add a symbol link linking to the SN4700 A1 sensor conf file to reuse.

#### How to verify it

Run sensor test against the SN4410 A1 system;
Run platform related regression test against the SN4410 A1 system
liat-grozovik pushed a commit that referenced this pull request Jul 29, 2022
…8379) (#11530)

- Why I did it
New SN410 A1 system has a different sensor layout with A0 system, needs a new sensor conf file to support it.

- How I did it
Since the SN4410 A1 system use exactly the same sensor layout as the SN4700 A1 system, so add a symbol link linking to the SN4700 A1 sensor conf file to reuse.

- How to verify it
Run sensor test against the SN4410 A1 system;
Run platform related regression test against the SN4410 A1 system
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.

5 participants