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] [202012] Enhance the platform.json with adding more platform device facts. #7496

Merged
merged 3 commits into from
May 9, 2021

Conversation

keboliu
Copy link
Collaborator

@keboliu keboliu commented Apr 30, 2021

Why I did it

Current platform.json lacks some peripheral device related facts, like chassis/fan/pasu/drawer/thermal/components names, numbers, etc.

How I did it

Add platform device facts to the platform.json file

How to verify it

Run sonic-mgmt platform API tests which depend on these facts.

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

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

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

@liat-grozovik
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik
Copy link
Collaborator

@jleveque please note that this PR is against 202012 and cannot be cherry picked from master as master has interfaces already included for dynamic port breakout which should not be taken to 202012.

@jleveque
Copy link
Contributor

jleveque commented May 3, 2021

/azp run

@liat-grozovik
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jleveque
Copy link
Contributor

jleveque commented May 7, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@keboliu
Copy link
Collaborator Author

keboliu commented May 8, 2021

Hi, @daall would you please help to merge to 202012?

@liat-grozovik liat-grozovik merged commit 100c140 into sonic-net:202012 May 9, 2021
@keboliu keboliu deleted the 202012-amend_platform_json branch May 10, 2021 02:26
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.

3 participants