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

[Disk Manager] check number of inflight tasks properly in test #2862

Merged
merged 7 commits into from
Jan 17, 2025

Conversation

gy2411
Copy link
Collaborator

@gy2411 gy2411 commented Jan 16, 2025

Test on inflight tasks limit should check the number of inflight tasks. This is not the same at the number of tasks in status 'running' because a task can be running but not inflight.

Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit bc764a9.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6323 6315 0 2 6 0

@gy2411 gy2411 force-pushed the users/gayurgin/dm_fix_test_tasks_running_limit branch from b88fc8e to 709b86c Compare January 16, 2025 14:22
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit a335742.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6322 6322 0 0 0 0

@gy2411 gy2411 force-pushed the users/gayurgin/dm_fix_test_tasks_running_limit branch from 4de7f99 to fb8a798 Compare January 17, 2025 17:50
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit e18a073.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6322 6322 0 0 0 0

@SvartMetal SvartMetal merged commit d73c679 into main Jan 17, 2025
11 of 12 checks passed
@SvartMetal SvartMetal deleted the users/gayurgin/dm_fix_test_tasks_running_limit branch January 17, 2025 21:57
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.

3 participants