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

sni_proxy.py: add option to select the temp directory used #640

Merged
merged 1 commit into from
Mar 9, 2025

Conversation

fruch
Copy link
Contributor

@fruch fruch commented Mar 9, 2025

since when we are working inside podman we can't mount /tmp and we need the exact same direcorty mount as host for sni_proxy to work as expected

since when we are working inside podman we can't mount /tmp
and we need the exact same direcorty mount as host for sni_proxy
to work as expected
@fruch fruch requested review from tchaikov and soyacz March 9, 2025 10:30
@mykaul
Copy link
Contributor

mykaul commented Mar 9, 2025

nit: SNI proxy was deprecated. Not sure it won't be back again in the future, but right now, deprecated.

@fruch
Copy link
Contributor Author

fruch commented Mar 9, 2025

nit: SNI proxy was deprecated. Not sure it won't be back again in the future, but right now, deprecated.

testing of it is still in dtest, and in all drivers.

@mykaul
Copy link
Contributor

mykaul commented Mar 9, 2025

nit: SNI proxy was deprecated. Not sure it won't be back again in the future, but right now, deprecated.

testing of it is still in dtest, and in all drivers.

We shouldn't.

@fruch fruch merged commit ef67958 into scylladb:next Mar 9, 2025
4 checks passed
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.

None yet

2 participants