Readme of PFCWD reboot and restart tests #2359
Conversation
docs/pfcwd/README_PFCWD_Restart.md
Outdated
| - [Background](#background) | ||
| - [PFC storm detection](#pfc-storm-detection) | ||
| - [PFC storm mitigation](#pfc-storm-mitigation) | ||
| - [PFC storm restoration](#pfc-storm-restoration) | ||
| - [PFC watchdog implementation](#pfc-watchdog-implementation) | ||
| - [PFC watchdog commands on SONiC](#pfc-watchdog-commands-on-sonic) |
There was a problem hiding this comment.
These sections can be omitted. Please move all this to a general readme under the pfcwd folder since it is the same for all cases
There was a problem hiding this comment.
Omitted the TOC from the readme.
|
|
||
| ## Experiment Steps | ||
|
|
||
| 1. Execute the test “PFCWD_Basic_All-to-all” test for only one priority (e.g., 3) |
There was a problem hiding this comment.
Please describe what the test "pfcwd basic all to all" does. Without that info, it is unclear what is being covered
There was a problem hiding this comment.
There was a problem hiding this comment.
Please provide a link to the testcase in this document
There was a problem hiding this comment.
PFC WD Test Plan is yet to merge in sonic-mgmt/master - is it ok to provide the link of PR ( #2205 ) in actual document.
| - [Background](#background) | ||
| - [PFC storm detection](#pfc-storm-detection) | ||
| - [PFC storm mitigation](#pfc-storm-mitigation) | ||
| - [PFC storm restoration](#pfc-storm-restoration) | ||
| - [PFC watchdog implementation](#pfc-watchdog-implementation) | ||
| - [PFC watchdog commands on SONiC](#pfc-watchdog-commands-on-sonic) |
There was a problem hiding this comment.
please move these sections to a general readme
There was a problem hiding this comment.
Omitted the TOC section from readme
| <code>$sudo warm-reboot [options] </code> | ||
|
|
||
| 2. Wait for the DUT to come up | ||
| 3. Execute the test "PFCWD\_Basic\_All-to-all" test for only one priority (e.g., 3) |
There was a problem hiding this comment.
Explain what is the pfcwd basic all to all test
There was a problem hiding this comment.
There was a problem hiding this comment.
Please provide a link to the testcase in this document
There was a problem hiding this comment.
PFC WD Test Plan is yet to merge in sonic-mgmt/master - is it ok to provide the link of PR ( #2205 ) in actual document ?
|
@neethajohn did author addressed all your questions? |
Summary:
Readme of PFCWD reboot and restart tests.
Type of change