Skip to content

TEST/MD: Add short sleep before retrying memory allocation - #10150

Merged
yosefe merged 1 commit into
openucx:masterfrom
yosefe:topic/test-md-add-short-sleep-before-retrying
Sep 18, 2024
Merged

TEST/MD: Add short sleep before retrying memory allocation#10150
yosefe merged 1 commit into
openucx:masterfrom
yosefe:topic/test-md-add-short-sleep-before-retrying

Conversation

@yosefe

@yosefe yosefe commented Sep 17, 2024 •
edited
Loading

Copy link
Copy Markdown
Member

Why

Fix occasional failures in test_md.alloc

2024-09-17T05:24:31.6460426Z [ RUN      ] ib/test_md.alloc/1 <mlx5_1>
2024-09-17T05:24:32.2106135Z /scrap/azure/agent-02/AZP_WORKSPACE/1/s/contrib/../test/gtest/uct/test_md.cc:460: Failure
2024-09-17T05:24:32.2108474Z Expected: ((double)num_alloc_failures / iterations) < (0.3), actual: 0.35 vs 0.3
2024-09-17T05:24:32.2110905Z Too many OUT_OF_RESOURCE failures
2024-09-17T05:24:32.2146490Z [  FAILED  ] ib/test_md.alloc/1, where GetParam() = mlx5_1 (570 ms)

@yosefe
TEST/MD: Add short sleep before retrying memory allocation
410a90a 
Resolve test failures in test_md.alloc by waiting after allocation
failure (in case another process is using the MEMIC) and change the
success threshold from 0.7 to 0.5.
@yosefe
yosefe enabled auto-merge September 17, 2024 09:01
@yosefe
yosefe merged commit 049eed4 into openucx:master Sep 18, 2024
@yosefe
yosefe deleted the topic/test-md-add-short-sleep-before-retrying branch September 18, 2024 12:43
This was referenced Aug 31, 2026
Merged
Merged
Merged
@svc-ucx svc-ucx mentioned this pull request Sep 11, 2026
Open
@svc-ucx svc-ucx mentioned this pull request Sep 21, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tvegas1 tvegas1 tvegas1 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@yosefe @tvegas1