validator: enable authorization by default - #549
Merged
ewienik merged 1 commit intoAug 10, 2026
Conversation
QuerthDP
force-pushed
the
enable-authorization-in-validator
branch
from
August 10, 2026 08:35
e8c0d0e to
18e2103
Compare
QuerthDP
marked this pull request as ready for review
August 10, 2026 08:37
There was a problem hiding this comment.
Pull request overview
Enables authenticated, reduced-privilege ScyllaDB access throughout the validator harness.
Changes:
- Adds default superuser bootstrap and restricted Vector Store role.
- Authenticates validator CQL sessions as superuser.
- Preserves authentication in Alternator and HA configurations.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
crates/validator/src/common.rs |
Adds default authentication and role setup. |
crates/validator/src/high_availability.rs |
Enables authentication for the HA test. |
crates/validator/src/alternator/mod.rs |
Retains default auth unless explicitly overridden. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
ewienik
previously approved these changes
Aug 10, 2026
Enable ScyllaDB authorization (PasswordAuthenticator + CassandraAuthorizer) by default across the validator harness, so the e2e suite exercises the same reduced-privilege setup used in production (Cloud) deployments instead of full/unauthenticated access. The default ScyllaDB cluster config now enables auth, and the Vector Store connects using a dedicated role granted only VECTOR_SEARCH_INDEXING and TEXT_SEARCH_INDEXING (mirroring the fine-grained indexing permissions used in Cloud rather than full SELECT). The validator's own test-driver CQL sessions keep connecting as an internal superuser so existing test setup/assertions are unaffected. Dedicated auth tests already manage their own isolated superuser/role setup and are unaffected, since the default bootstrap only runs when a cluster was started with the shared default auth config. Fixes: VECTOR-758
QuerthDP
force-pushed
the
enable-authorization-in-validator
branch
from
August 10, 2026 10:34
18e2103 to
687e73a
Compare
Member
Author
|
Changelog:
|
ewienik
approved these changes
Aug 10, 2026
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
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.
Enable ScyllaDB authorization (PasswordAuthenticator + CassandraAuthorizer) by default across the validator harness, so the e2e suite exercises the same reduced-privilege setup used in production (Cloud) deployments
instead of full/unauthenticated access.
The default ScyllaDB cluster config now enables auth, and the Vector Store connects using a dedicated role granted only
VECTOR_SEARCH_INDEXINGandTEXT_SEARCH_INDEXING(mirroring the fine-grained indexing permissions used in Cloud rather than fullSELECT). The validator's own test-driver CQL sessions keep connecting as an internal superuser so existing test setup/assertions are unaffected.Dedicated auth tests already manage their own isolated superuser/role setup and are unaffected, since the default bootstrap only runs when a cluster was started with the shared default auth config.
Fixes: VECTOR-758