This repository has been archived by the owner on Jul 22, 2024. It is now read-only.
Fixed hcdiag chk-ib-pcispeed test for RHEL 8.4 #1017 #1022
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR contains a fix and testcase enhancement for #1017
I was able to reproduce the problem described in the original issue. I could not reproduce the problem on RHEL 7.6-Alt, it was introduced somewhere between RHEL 7.6-Alt and RHEL 8.4. I validated that the fix works successfully on both RHEL 7.6-Alt and RHEL 8.4.
Unit test of
chk-ib-pcispeed
on RHEL 8.4, before the fix (failure case):Unit test of
chk-ib-pcispeed
on RHEL 8.4, after the fix (verifies the fix):Unit test of
chk-ib-pcispeed
on RHEL 7.6-Alt, before the fix (issue does not occur):[root@c650mnp06 ~]# xdsh c650f02p13 "cat /etc/redhat-release"
Unit test of
chk-ib-pcispeed
on RHEL 7.6-Alt, after the fix (verifies no regression introduced by the fix):In addition to testing the hcdiag chk-ib-pcispeed test, I also added a new FVT testcase to cover this feature during regular regression testing.
Unit test of FVT hcdiag test case on RHEL 8.4, before the fix (shows the test failing):
Unit test of FVT hcdiag test case on RHEL 8.4, after the fix (verifies the test is successful):