Skip to content

bump Rust Driver to 1.8.0 - #496

Merged
wprzytula merged 2 commits into
scylladb:masterfrom
wprzytula:bump-rust-driver-1.8.0
Aug 20, 2026
Merged

bump Rust Driver to 1.8.0#496
wprzytula merged 2 commits into
scylladb:masterfrom
wprzytula:bump-rust-driver-1.8.0

Conversation

@wprzytula

Copy link
Copy Markdown
Contributor

As a result of the bump, some tests from ControlConnectionTests broke, because some log messages changed. The tests are adjusted for the new contents.

Pre-review checklist

  • I have split my patch into logically separate commits.
  • All commit messages clearly explain what they change and why.
  • PR description sums up the changes and reasons why they should be introduced.
  • [ ] I have provided docstrings for the public items that I want to introduce.
  • [ ] I have adjusted the documentation in ./docs/source/.
  • [ ] I have implemented Rust unit tests for the features/changes introduced.
  • [ ] I have enabled appropriate tests in Makefile in {SCYLLA,CASSANDRA}_(NO_VALGRIND_)TEST_FILTER.
  • [ ] I added appropriate Fixes: annotations to PR description.

ControlConnectionTests broke because some log messages changed. The
tests are adjusted for the new contents.
@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: QUIET

Plan: Pro Plus

Run ID: 3c847303-8544-49cf-a128-d1c75fdaa764

📥 Commits

Reviewing files that changed from the base of the PR and between 9c75520 and 63fe5c3.

📒 Files selected for processing (1)
  • tests/src/integration/tests/test_control_connection.cpp
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • scylladb/scylladb (auto-detected)

Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review.


📝 Walkthrough

Walkthrough

The Scylla Rust runtime and development dependencies now use the v1.8.0 tag. Three invalid-connection integration tests now expect specific control-connection establishment failure messages that include the configured address.

Possibly related PRs

Suggested reviewers: lorak-mmk

Merge Risk: ⚪ Minimal · up to 63fe5

This change updates the Rust driver and aligns affected control-connection test expectations with the new log messages; no actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed The description explains the dependency upgrade, the resulting test changes, and the completed checklist items.
Title check ✅ Passed The title clearly and concisely identifies the primary change: upgrading the Rust driver to version 1.8.0.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Comment @coderabbitai help to get the list of available commands.

@wprzytula wprzytula added this to the 1.2 milestone Aug 20, 2026
@wprzytula wprzytula self-assigned this Aug 20, 2026
@wprzytula
wprzytula requested review from Lorak-mmk and a balanced review from Copilot August 20, 2026 15:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Rust driver dependency to v1.8.0 and aligns control-connection tests with its revised logging.

Changes:

  • Bumps Scylla Rust crates to v1.8.0.
  • Refreshes locked package versions and commit references.
  • Updates expected control-connection log messages.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
scylla-rust-wrapper/Cargo.toml Updates Rust driver dependencies.
scylla-rust-wrapper/Cargo.lock Locks v1.8.0 dependency versions.
tests/src/integration/tests/test_control_connection.cpp Adjusts expected log messages.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread tests/src/integration/tests/test_control_connection.cpp
As Copilot noted, this unbreaks tests on remote CCM deployments, whatever they are.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@wprzytula

Copy link
Copy Markdown
Contributor Author

Merging even though DEB packages CI job is failing. This is because server to download openssl from returns 502/503, which means temporary server error.

@wprzytula
wprzytula merged commit cd7841f into scylladb:master Aug 20, 2026
12 of 14 checks passed
@wprzytula
wprzytula deleted the bump-rust-driver-1.8.0 branch August 20, 2026 16:06
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.

3 participants