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 ixs7215] Add new platform capabilities to platform.json #9032

Merged
merged 1 commit into from
Oct 27, 2021

Conversation

dflynn-Nokia
Copy link
Contributor

This commit more fully declares the HW capabilities of the Nokia-7215
platform. For example, support for the threshold values associated with each
thermal sensor is described. The intent here is to inform the sonic-mgmt
platform test cases of which HW features are supported.

This commit must align with PR# 4521 within the sonic-mgmt git repo which is
currently under review. Any changes to that PR will need to be reflected in
this commit.

Why I did it

Eliminate sonic-mgmt test case failures within test cases that attempt to verify HW features that are not supported by the Nokia-7215 platform.

How I did it

Add capabilities to the platform.json file to describe which HW features are supported.

How to verify it

Run the sonic-mgmt platform test suite

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)

This commit more fully declares the HW capabilities of the Nokia-7215
platform. For example, support for the threshold values associated with each
thermal sensor is described. The intent here is to inform the sonic-mgmt
platform test cases of which HW features are supported.

This commit must align with PR# 4521 within the sonic-mgmt git repo which is
currently under review. Any changes to that PR will need to be reflected in
this commit.
@Blueve
Copy link
Contributor

Blueve commented Oct 22, 2021

@dflynn-Nokia Do you want to merge this PR right now?

@dflynn-Nokia
Copy link
Contributor Author

My thinking is that we should hold off on merging this PR until agreement is reached on sonic-mgmt PR# https://github.com/Azure/sonic-mgmt/pull/4521

@Blueve
Copy link
Contributor

Blueve commented Oct 26, 2021

sonic-net/sonic-mgmt#4521 @dflynn-Nokia the sonic-mgmt PR has been merged, can we merge this one now?

@dflynn-Nokia
Copy link
Contributor Author

Yes, now that sonic-mgmt PR 4521 is merged, please merge and backport this PR.

@Blueve Blueve merged commit 1957558 into sonic-net:master Oct 27, 2021
qiluo-msft pushed a commit that referenced this pull request Oct 27, 2021
This commit more fully declares the HW capabilities of the Nokia-7215
platform. For example, support for the threshold values associated with each
thermal sensor is described. The intent here is to inform the sonic-mgmt
platform test cases of which HW features are supported.

This commit must align with PR# 4521 within the sonic-mgmt git repo which is
currently under review. Any changes to that PR will need to be reflected in
this commit.
judyjoseph pushed a commit that referenced this pull request Nov 14, 2021
This commit more fully declares the HW capabilities of the Nokia-7215
platform. For example, support for the threshold values associated with each
thermal sensor is described. The intent here is to inform the sonic-mgmt
platform test cases of which HW features are supported.

This commit must align with PR# 4521 within the sonic-mgmt git repo which is
currently under review. Any changes to that PR will need to be reflected in
this commit.
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.

4 participants