Fix test_client_routes_change_event #656
Conversation
sylwiaszunejko
force-pushed
the
fix-test-change-routes
branch
from
dc737f8 to
55a05fd
Compare
|
@Lorak-mmk There is some problem with this test, it uses |
| runs-on: ubuntu-24.04 | ||
| env: | ||
| SCYLLA_VERSION: release:2025.2 | ||
| SCYLLA_VERSION: release:2026.1 |
There was a problem hiding this comment.
my mistake, I saw it on driver matrix and locally with this setting I was able to successfully run this test
There was a problem hiding this comment.
what is the first version that supports CLIENT_ROUTES_CHANGE?
There was a problem hiding this comment.
ok, it is 2026.1.0, so we need to use it at least for testing purposes in the PR, and remove the commit that bums the scylla version, before the merge, is that ok?
There was a problem hiding this comment.
just run this test locally
There was a problem hiding this comment.
locally it works just fine
| runs-on: ubuntu-24.04 | ||
| env: | ||
| SCYLLA_VERSION: release:2025.2 | ||
| SCYLLA_VERSION: release:2026.1 |
There was a problem hiding this comment.
just run this test locally
sylwiaszunejko
force-pushed
the
fix-test-change-routes
branch
from
55a05fd to
8adfac8
Compare
|
@dkropachev please take a look |
sylwiaszunejko
force-pushed
the
fix-test-change-routes
branch
from
8adfac8 to
72eb00d
Compare
4 participants
Bumped SCYLLA_VERSION to verify that the test is working as expected.
Fixes: #652