Skip to content
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

[Backport 2.x] [Version] Don't spoof major for 3.0+ clusters #2750

Merged
merged 1 commit into from
Apr 4, 2022

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 2d716ad from #2722

@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team as a code owner April 4, 2022 19:03
@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success e16d8661688dfdb89bae910508f9c4dd6d7b8964
Log 4143

Reports 4143

@nknize nknize force-pushed the backport/backport-2722-to-2.x branch from e16d866 to 51c0817 Compare April 4, 2022 20:27
Changes version comparison logic to only translate major version when
comparing with legacy 7x versions. This is needed beginning in 2.0 so
that when running 2.0+ versions in bwc mode for 3.0+ upgrades, node
versions no longer have to translate major version or spoof to legacy
versions.

Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
(cherry picked from commit 2d716ad)
@nknize nknize force-pushed the backport/backport-2722-to-2.x branch from 51c0817 to 1c215a7 Compare April 4, 2022 20:43
@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success 51c0817799ca8c120e37c64a69dd01fa8b05f2df
Log 4146

Reports 4146

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success 1c215a7
Log 4148

Reports 4148

@nknize nknize merged commit debb45c into 2.x Apr 4, 2022
@github-actions github-actions bot deleted the backport/backport-2722-to-2.x branch April 4, 2022 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants