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

csi_driver: add host param to every cmdline endpoint to allow overriding of localhost #2554

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

zeil
Copy link
Contributor

@zeil zeil commented Nov 22, 2024

This allows to run nbs and csi_driver in local kubernetes cluster via kind tool (it uses different network namespaces and bridging under the hood).

Copy link
Contributor

Hi! Thank you for contributing!
The tests on this PR will run after a maintainer adds an ok-to-test label to this PR manually. Thank you for your patience!

@antonmyagkov antonmyagkov added the ok-to-test Label to approve test launch for external members label Nov 22, 2024
@antonmyagkov antonmyagkov self-requested a review November 22, 2024 16:12
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Nov 22, 2024
@antonmyagkov antonmyagkov added blockstore Add this label to run only cloud/blockstore build and tests on PR ok-to-test Label to approve test launch for external members labels Nov 22, 2024
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Nov 22, 2024
Copy link
Contributor

Note

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

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

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

@antonmyagkov antonmyagkov merged commit 96b1794 into ydb-platform:main Nov 25, 2024
25 of 27 checks passed
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants