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

[rdma] Test PFC watchdog's impact on runtime traffic #2979

Merged
merged 4 commits into from
Feb 18, 2021

Conversation

baiwei0427
Copy link
Contributor

Description of PR

Signed-off-by: Wei Bai [email protected]

Summary: Test PFC watchdog's impact on runtime traffic
Fixes # (issue)

Type of change

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

Approach

What is the motivation for this PR?

Test if enabling PFC watchdog will impact runtime traffic

How did you do it?

Start data traffic first then enable PFC watchdog at the SONiC DUT

How did you verify/test it?

I did test using Mellanox SN2700 and IXIA chassis

Any platform specific information?

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

Currently the test case only supports T0 topology

Documentation

@baiwei0427 baiwei0427 requested a review from a team as a code owner February 11, 2021 08:37
@baiwei0427 baiwei0427 changed the title [rdma] Pfcwd runtime traffic dev [rdma] Test PFC watchdog's impact on runtime traffic Feb 11, 2021
@lgtm-com
Copy link

lgtm-com bot commented Feb 11, 2021

This pull request introduces 2 alerts when merging 7e18fb8 into ad98413 - view on LGTM.com

new alerts:

  • 2 for Unused import

Copy link
Contributor

@neethajohn neethajohn left a comment

Choose a reason for hiding this comment

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

Please fix the LGTM alerts as well

@neethajohn neethajohn merged commit e893f73 into sonic-net:master Feb 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants