[8.10] Backport Exclude desired_nodes validation tests from mixed-cluster QA#101421
[8.10] Backport Exclude desired_nodes validation tests from mixed-cluster QA#101421ldematte merged 1 commit intoelastic:8.10from
Conversation
| // the nodes on current version. The rest client then round robins | ||
| // between the two nodes on old version. In order to unmute this, | ||
| // we need a different node selector, that always consistently | ||
| // selects the same node. |
There was a problem hiding this comment.
We've actually tested this, it doesn't seem like to be the case.. Have you tried version skips for these tests and you still see them running in bwc tests?
There was a problem hiding this comment.
Oh nevermind, this is to exclude the tests entirely.
There was a problem hiding this comment.
Yes exactly. In mixed clusters you start with a version and then upgrade it to current; you may talk to nodes on different versions, non predictably, and end up with different behaviour based on which node you are talking to.
Also, the version skip is already there and works well in non-mixed scenarios, but for these tests it seems to apply to the "start" (older) version, so once upgraded you have "worked around" the skip.
I think I've see in other tests that the version is read from the node(s) and a suitable node is selected, but it does not seem like this case.
Backport of #100682 to 8.10
Closes #101419