cluster: use dynamic whitelist policy for proxy access#833
Closed
dkropachev wants to merge 14 commits into
Closed
Conversation
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
dkropachev
force-pushed
the
dk/fix-for-proxy-access-2
branch
from
679a13f to
ab38a22
Compare
dkropachev
force-pushed
the
dk/fix-for-proxy-access-2
branch
from
ab38a22 to
3b6c175
Compare
dkropachev
force-pushed
the
dk/fix-for-proxy-access-2
branch
from
3b6c175 to
a4147e2
Compare
|
I can confirm taking this PR in scylladb/scylla-cluster-tests#14512, and it worked as expected via public address |
dkropachev
force-pushed
the
dk/fix-for-proxy-access-2
branch
from
fab5d7b to
be1794d
Compare
dkropachev
force-pushed
the
dk/fix-for-proxy-access-2
branch
from
be1794d to
589a752
Compare
| @mock_session_pools | ||
| def test_session_preserves_down_event_discounting_after_endpoint_update(self, *_): | ||
| @total_ordering | ||
| class _DeterministicHashEndPoint(EndPoint): |
fruch
added a commit
to fruch/scylla-cluster-tests
that referenced
this pull request
May 11, 2026
…ress connections Switch python-driver to PR scylladb/python-driver#833 branch which introduces DynamicWhiteListRoundRobinPolicy. When SCT connects via public addresses (IP_SSH_CONNECTIONS == "public"), use this policy instead of WhiteListRoundRobinPolicy. DynamicWhiteListRoundRobinPolicy dynamically tracks which host the control connection resolves to and only routes queries there, solving the problem where the driver discovers private IPs from system tables that are unreachable from the client when using public/proxy access.
Collaborator
Author
|
Change is too intrusive, given pre-existing mess in node status handling it will only contribute to it, we found easier solution - #878, this PR is closed |
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pre-review checklist
./docs/source/.Fixes:annotations to PR description.