Skip to content

[platform_test] Skip test_get_high_critical_threshold for Nokia 7215#5791

Merged
Blueve merged 1 commit intosonic-net:masterfrom
Blueve:dev/jika/skip_cth_7215
Jun 10, 2022
Merged

[platform_test] Skip test_get_high_critical_threshold for Nokia 7215#5791
Blueve merged 1 commit intosonic-net:masterfrom
Blueve:dev/jika/skip_cth_7215

Conversation

@Blueve
Copy link
Collaborator

@Blueve Blueve commented Jun 9, 2022

Description of PR

Summary:
Skip test_get_high_critical_threshold for Nokia 7215

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012

Approach

What is the motivation for this PR?

Skip test_get_high_critical_threshold for Nokia 7215.
The platform does not support high critical threshold according to PR: sonic-net/sonic-buildimage#10533

How did you do it?

Update conditional marker to skip it

How did you verify/test it?

Run test with Nokia 7215 and the test been skipped

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@Blueve Blueve requested a review from a team as a code owner June 9, 2022 07:50
Copy link
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM

@Blueve Blueve merged commit 69d7a2c into sonic-net:master Jun 10, 2022
@Blueve Blueve deleted the dev/jika/skip_cth_7215 branch June 10, 2022 00:42
wangxin pushed a commit that referenced this pull request Jun 10, 2022
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