scylladb / scylla-cluster-tests

Tests for Scylla Clusters
GNU Affero General Public License v3.0
58 stars 95 forks source link

fix(raft): drop stale log pattern as trigger for interrupting decommission #9340

Open lersek opened 3 days ago

lersek commented 3 days ago

Since https://github.com/scylladb/scylladb/commit/36520250623a, scylla does not log "Finished token ring movement". Watching the log for this message would inevitably time out, consequently. Remove this log pattern.

Fixes #9281

The backporting question looks pretty bleak on this one:

This means that the patch should be backported minimally to branch-5.3 through branch-6.2 (because I presume those target identically-versioned, released scylla binaries, and those binaries no longer emit the message).

I don't know how the other SCT branch names (i.e., those that include a year number, or the word "perf") relate to scylla binary 5.3. Thus, those branches may or may not be backporting targets.

fruch commented 2 days ago

branch 5.4 isn't supported anymore.

and we should label the enterprise branches in SCT

lersek commented 23 hours ago

OK, thanks.

In enterprise, https://github.com/scylladb/scylladb/commit/36520250623a is directly part of branches branch-2024.1 and branch-2024.2.

Furthermore, the change had also been incorporated into branch-2023.1, through https://github.com/scylladb/scylladb/commit/ea506f50cca3.

lersek commented 23 hours ago

Ah I misunderstood; you didn't mean we shouldn't label open source! Restoring those labels now.