-
Notifications
You must be signed in to change notification settings - Fork 25.5k
[ML] Fix Transform upgrade test #132995
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ML] Fix Transform upgrade test #132995
Conversation
Pinging @elastic/ml-core (Team:ML) |
|
||
// do not allow in mixed clusters | ||
if (TransportVersionUtils.isMinTransportVersionSameAsCurrent(state) == false) { | ||
if (state.nodes().isMixedVersionCluster()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The upgrade test failed because the original check for a mixed cluster isMinTransportVersionSameAsCurrent
depends on the order in which the nodes are upgraded. If the master node this code runs on has not been upgraded yet the test will pass.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice find
.../qa/rolling-upgrade/src/test/java/org/elasticsearch/upgrades/TransformSurvivesUpgradeIT.java
Show resolved
Hide resolved
The upgrade tests from 8.19.5 are failing because the transform upgrade code is fighting it out with the automatic system index upgrader. First the system index migrator moves
Then
|
|
||
// do not allow in mixed clusters | ||
if (TransportVersionUtils.isMinTransportVersionSameAsCurrent(state) == false) { | ||
if (state.nodes().isMixedVersionCluster()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice find
The test failed comparing an int to a long
Closes #132892