bump Rust Driver to 1.8.0 - #496
Conversation
ControlConnectionTests broke because some log messages changed. The tests are adjusted for the new contents.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: QUIET Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review. 📝 WalkthroughWalkthroughThe Scylla Rust runtime and development dependencies now use the Possibly related PRs
Suggested reviewers: Merge Risk: ⚪ Minimal · up to 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)
Comment |
There was a problem hiding this comment.
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.
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>
|
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. |
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 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 inMakefilein{SCYLLA,CASSANDRA}_(NO_VALGRIND_)TEST_FILTER.[ ] I added appropriateFixes:annotations to PR description.