Skip to content

validator: enable authorization by default - #549

Merged
ewienik merged 1 commit into
scylladb:masterfrom
QuerthDP:enable-authorization-in-validator
Aug 10, 2026
Merged

validator: enable authorization by default#549
ewienik merged 1 commit into
scylladb:masterfrom
QuerthDP:enable-authorization-in-validator

Conversation

@QuerthDP

@QuerthDP QuerthDP commented Aug 7, 2026

Copy link
Copy Markdown
Member

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
QuerthDP force-pushed the enable-authorization-in-validator branch from e8c0d0e to 18e2103 Compare August 10, 2026 08:35
@QuerthDP
QuerthDP marked this pull request as ready for review August 10, 2026 08:37
@QuerthDP
QuerthDP requested review from ewienik and a balanced review from Copilot August 10, 2026 08:37

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
ewienik previously approved these changes Aug 10, 2026
Comment thread crates/validator/src/common.rs Outdated
Comment thread crates/validator/src/common.rs
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
QuerthDP force-pushed the enable-authorization-in-validator branch from 18e2103 to 687e73a Compare August 10, 2026 10:34
@QuerthDP

Copy link
Copy Markdown
Member Author

Changelog:

  • added use_default_auth bool param to the init_with_config
  • deduplicated the SUPERUSER setup from auth.rs and alternator/auth.rs

@QuerthDP
QuerthDP requested a review from ewienik August 10, 2026 10:41
@ewienik
ewienik added this pull request to the merge queue Aug 10, 2026
Merged via the queue into scylladb:master with commit a17073b Aug 10, 2026
44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants