fabtests/efa: Improve MR exhaustion test #9676
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
wenduwan
reviewed
Dec 27, 2023
a-szegel
approved these changes
Dec 28, 2023
|
Make sure to grab Wenduo's comment, also looks like there are some real fabtest failures here. |
|
bot:aws:retest |
sunkuamzn
force-pushed
the
mr-exhaust-unexp
branch
from
f61970e to
66ca51a
Compare
wenduwan
approved these changes
Dec 28, 2023
A simple implementation of the unexpected pingpong test that does not do inject and only sends one unexpected message at a time Signed-off-by: Sai Sunku <sunkusa@amazon.com>
It's better to query the device than to hardcode the limit in case the limit changes in the future Signed-off-by: Sai Sunku <sunkusa@amazon.com>
Signed-off-by: Sai Sunku <sunkusa@amazon.com>
sunkuamzn
force-pushed
the
mr-exhaust-unexp
branch
from
66ca51a to
8c4731d
Compare
|
Fixed fabtest failures. CI passing now. |
wenduwan
approved these changes
Dec 29, 2023
This was referenced Jan 2, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Some improvements to the MR exhaustion test