Skip to content

[NBS] Include all checked paths in volume resolution errors - #7607

Merged
drbasic merged 3 commits into
mainfrom
users/drbasic/describe-volume-checked-paths
Oct 26, 2026
Merged

drbasic merged 3 commits into
mainfrom
users/drbasic/describe-volume-checked-paths

Conversation

@drbasic

@drbasic drbasic commented Oct 23, 2026

Copy link
Copy Markdown
Collaborator

Notes

When a volume cannot be found, the error previously reported only the last checked path, often ending in -copy. This could misleadingly suggest that the request targeted a disk copy.
The error now includes all checked paths in search order. Individual attempts omit the single-path suffix, and TDescribeVolumeActor adds the complete list to the final error.

Was:

BLOCKSTORE_VOLUME_PROXY ERROR: [vp:? d:disk pg:0 t:895us] Could not resolve path for volume. Error: SEVERITY_ERROR | FACILITY_SCHEMESHARD | 2 Path not found, path="/NBS/_257/disk-copy" f@silent

Now:

BLOCKSTORE_VOLUME_PROXY ERROR: [vp:? d:disk pg:0 t:895us] Could not resolve path for volume. Error: SEVERITY_ERROR | FACILITY_SCHEMESHARD | 2 Path not found, checked paths=["/NBS/disk", "/NBS/_abc/disk", "/NBS/_257/disk-copy"] f@silent

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

github-actions Bot commented Oct 23, 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.

  • 🚫 blockstore (cancelled or timed out before reporting completion)

qkrorlqr
qkrorlqr previously approved these changes Oct 23, 2026
@github-actions

github-actions Bot commented Oct 23, 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: 11468s): some tests FAILED for commit 80c9d5c.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
8185 8155 0 3 0 27 0

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

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
6 3 0 3 0 0 0

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

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
6 3 0 3 0 0 0

qkrorlqr
qkrorlqr previously approved these changes Oct 23, 2026
@github-actions

github-actions Bot commented Oct 23, 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.

  • 🚫 blockstore (cancelled or timed out before reporting completion)

@drbasic
drbasic force-pushed the users/drbasic/describe-volume-checked-paths branch from 9b65509 to f96c243 Compare October 23, 2026 12:45
@github-actions

github-actions Bot commented Oct 23, 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: 11152s): all tests PASSED for commit f96c243.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
8185 8158 0 0 0 27 0

@drbasic
drbasic merged commit d92e8b7 into main Oct 26, 2026
16 of 17 checks passed
@drbasic
drbasic deleted the users/drbasic/describe-volume-checked-paths branch October 26, 2026 10:29
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.

3 participants