ci: Pin scylladb docker to fix integration tests#3126
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #3126 +/- ##
========================================
Coverage 82.61% 82.61%
========================================
Files 397 397
Lines 16034 16034
Branches 8399 8399
========================================
+ Hits 13246 13247 +1
Misses 1667 1667
+ Partials 1121 1120 -1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
godexsoft
left a comment
There was a problem hiding this comment.
We should be careful with scylla and their driver.. we have seen that updating the driver also broke things last time we tried. My understanding is that this change is pinning the older version, right?
Yes, it's pinning the version before the latest release, which broke something. |
It seems that new scylladb versions have changed
SimpleStrategythat we use and now code fails with:This should fix it for now in integration tests, but we should take a closer look if we need to migrate the database