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.

Sign up for GitHub

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

Jump to bottom

Return E_NOT_FOUND error if volume/filestore doesn't exist #2718

Open
antonmyagkov opened this issue Dec 17, 2024 · 0 comments
Open

Return E_NOT_FOUND error if volume/filestore doesn't exist #2718

antonmyagkov opened this issue Dec 17, 2024 · 0 comments
Assignees
@antonmyagkov
Labels
blockstore Add this label to run only cloud/blockstore build and tests on PR filestore Add this label to run only cloud/filestore build and tests on PR

Comments

@antonmyagkov
Copy link
Collaborator

antonmyagkov commented Dec 17, 2024 •
edited
Loading

Return E_NOT_FOUND error instead of NKikimrScheme::StatusPathDoesNotExist if volume/filestore doesn't exist

https://github.com/ydb-platform/nbs/blob/main/cloud/blockstore/libs/storage/ss_proxy/ss_proxy_actor_describevolume.cpp#L152

We must support both error codes in error handlers for backward compatibility
@antonmyagkov antonmyagkov added blockstore Add this label to run only cloud/blockstore build and tests on PR filestore Add this label to run only cloud/filestore build and tests on PR labels Dec 17, 2024
@antonmyagkov antonmyagkov self-assigned this Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@antonmyagkov antonmyagkov

Labels
blockstore Add this label to run only cloud/blockstore build and tests on PR filestore Add this label to run only cloud/filestore build and tests on PR
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@antonmyagkov