Skip to content

[Blockstore] split local nonrepl loadtests by backend - #6861

Open
kobzonega wants to merge 2 commits into
mainfrom
nbs-7564-split-local-nonrepl-backends
Open

[Blockstore] split local nonrepl loadtests by backend#6861
kobzonega wants to merge 2 commits into
mainfrom
nbs-7564-split-local-nonrepl-backends

Conversation

@kobzonega

@kobzonega kobzonega commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Notes

Allows the io_uring test target to be configured independently

  • add variable NBS_LOCAL_NONREPL_BACKEND to select backend in the local-nonrepl test
  • add a separate embedded-io-uring target
  • register the new target in RECURSE_FOR_TESTS
  • keep the existing embedded target focused on aio

Issue

Sandbox tasks run tests on an older Linux kernel (<6.12) by default. The io_uring tests require features and fixes from a newer kernel, so enabling them in Arcadia also requires infrastructure changes that provide a suitable VM and kernel environment. This PR contains only the common parts that can be upstreamed to GitHub

@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Note

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

Note

All workloads for linux-x86_64-relwithdebinfo have completed.

Tip

Planned checks for linux-x86_64-relwithdebinfo.

  • 🚫 tasks + storage (cancelled or timed out before reporting completion)
  • 🚫 blockstore (cancelled or timed out before reporting completion)
  • 🚫 disk + manager (cancelled or timed out before reporting completion)
  • 🚫 filestore (cancelled or timed out before reporting completion)

@kobzonega kobzonega added blockstore Add this label to run only cloud/blockstore build and tests on PR large-tests Launch large tests for PR labels Aug 21, 2026
@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Note

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

Note

All workloads for linux-x86_64-relwithdebinfo have completed.

Tip

Planned checks for linux-x86_64-relwithdebinfo.

🔴 linux-x86_64-relwithdebinfo target: cloud/blockstore/ (test time: 12376s): some tests FAILED for commit f792a52.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
7530 7528 0 1 0 1 0

🟢 linux-x86_64-relwithdebinfo target: cloud/blockstore/ (test time: 296s): all tests PASSED for commit f792a52.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
2 2 0 0 0 0 0

RECURSE_FOR_TESTS(
dedicated
embedded
embedded-io-uring

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about dedicated tests ... they are using uring as well

SELECTED_BACKEND = os.getenv('NBS_LOCAL_NONREPL_BACKEND')
if SELECTED_BACKEND:
BACKENDS = {SELECTED_BACKEND: BACKENDS[SELECTED_BACKEND]}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe just detect kernel version and skip io_uring test if kernel version is lower then min version required

@github-actions

github-actions Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Note

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

Important

Workload for linux-x86_64-relwithdebinfo is not finished yet. This comment will be updated after all workloads complete.

Tip

Planned checks for linux-x86_64-relwithdebinfo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blockstore Add this label to run only cloud/blockstore build and tests on PR large-tests Launch large tests for PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants