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

prov/cxi: ignore FLT_OVERFLOW and FLT_INVALID errors #10513

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

heemstra
Copy link
Contributor

@heemstra heemstra commented Nov 1, 2024

Don't treat CXIP_COLL_RC_FLT_OVERFLOW and CXIP_COLL_RC_FLT_INVALID as errors. Add logging for when one of these two or CXIP_COLL_RC_FLT_INEXACT is encountered.

Don't treat CXIP_COLL_RC_FLT_OVERFLOW and CXIP_COLL_RC_FLT_INVALID
as errors. Add logging for when one of these two or CXIP_COLL_RC_FLT_INEXACT
is encountered.

Signed-off-by: John Heemstra <heemstra@hpe.com>
@heemstra
Copy link
Contributor Author

heemstra commented Nov 5, 2024

@j-xiong @shijin-aws - can you comment on these failures?

@shijin-aws
Copy link
Contributor

bot:aws:retest

@shijin-aws
Copy link
Contributor

AWS CI failure is unrelated. can ignore

@j-xiong
Copy link
Contributor

j-xiong commented Nov 5, 2024

There is one failed test with mpich over tcp. I don't think it is related.

@swelch swelch merged commit a03c5a7 into ofiwg:main Nov 5, 2024
11 of 13 checks passed
@heemstra heemstra deleted the ignore_ovf_inv branch November 6, 2024 14:18
heemstra added a commit to heemstra/libfabric that referenced this pull request Nov 12, 2024
Update the unit tests for collectives to account for previously-merged
PR: ofiwg#10513
heemstra added a commit to heemstra/libfabric that referenced this pull request Nov 12, 2024
Update the unit tests for collectives to account for previously-merged
PR: ofiwg#10513

Signed-off-by: John Heemstra <heemstra@hpe.com>
iziemba pushed a commit that referenced this pull request Nov 13, 2024
Update the unit tests for collectives to account for previously-merged
PR: #10513

Signed-off-by: John Heemstra <heemstra@hpe.com>
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.

4 participants