Classic config peer discovery still might select two different seed nodes

[dcorbacho]

Describe the bug

Sometimes in CI the testcase peer_discovery_classic_config_SUITE:successfully_discovery fails to cluster all 7 nodes. Six nodes join the same seed, and one other node selects itself as a seed and does not join the rest of the cluster.

The peer discovery seed selection needs to be reviewed

Reproduction steps

1. It's a CI flake

Expected behavior

The 7 nodes join the cluster

Additional context

No response

[michaelklishin]

This was discussed relatively recently. Given that for this backend, the list of nodes is explicitly listed ahead of time, should we make the seed explicitly configurable, and use it in the tests?

We have tried different approaches over the years, such as "use the first node on a sorted list".

For example, Cassandra uses an explicit seed node configuration argument.

@dumbbell @mkuratczyk

[dumbbell]

@dcorbacho: Do you have a link to a failing CI job or node logs?