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

[Version] Don't spoof major for 3.0+ clusters #2722

Merged
merged 8 commits into from
Apr 4, 2022

Conversation

nknize
Copy link
Collaborator

@nknize nknize commented Apr 2, 2022

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.

@nknize nknize added non-issue bugs / unexpected behaviors that end up non issues; audit trail simple changes that aren't issues >upgrade Label used when upgrading library dependencies (e.g., Lucene) v3.0.0 Issues and PRs related to version 3.0.0 labels Apr 2, 2022
@nknize nknize requested a review from a team as a code owner April 2, 2022 19:44
@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure 4f0ec4e4a0f4d181fe9f2c629a5cfd17dda15c38
Log 4072

Reports 4072

@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure d1b10ff4780c25df94cde9c1c58c7928dfe2135b
Log 4073

Reports 4073

@nknize
Copy link
Collaborator Author

nknize commented Apr 2, 2022

Logic for supporting legacy versions remain; for now. Will remove in a follow up PR

@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure 1ec64b7e14c88c6fb72e590fa50f2963352ab278
Log 4074

Reports 4074

@nknize
Copy link
Collaborator Author

nknize commented Apr 3, 2022

#1703 repeat... refiring

@nknize
Copy link
Collaborator Author

nknize commented Apr 3, 2022

start gradle check

@nknize nknize force-pushed the versionBump/prep3.0 branch from 10e709b to 5e94cd4 Compare April 3, 2022 05:53
@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success 1ec64b7e14c88c6fb72e590fa50f2963352ab278
Log 4077

Reports 4077

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success 10e709bd72101e95545227d7461dbd08a6fca50f
Log 4078

Reports 4078

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success 5e94cd4efdfb580943a1dba663cbe0b1e1a99f0a
Log 4079

Reports 4079

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success 2cbc329ba813e989a52b06589da3407b93bfbebe
Log 4095

Reports 4095

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success cbd6e0dde3c65313a60ad5523c2b43ee9928d36a
Log 4096

Reports 4096

@nknize nknize changed the title [Version] Add initial 3.0 support logic [Version] Don't spoof major for 3.0+ clusters Apr 4, 2022
@nknize nknize mentioned this pull request Apr 4, 2022
@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success 06d9b4037b354a2f2068a93b46f9262d527f3d90
Log 4098

Reports 4098

nknize added 8 commits April 4, 2022 11:35
This adds initial testing to prep 3.0 for removal of legacy support

Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
….x and 2.x

Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
@nknize nknize force-pushed the versionBump/prep3.0 branch from 06d9b40 to 6e9bc40 Compare April 4, 2022 16:41
@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success 6e9bc40
Log 4125

Reports 4125

// all major versions from 3 onwards are compatible with last minor series of the previous major
// todo: remove 5 check when removing LegacyESVersionTests
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we cut an issue for this removal in 3.x?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 #2748

@nknize nknize merged commit 2d716ad into opensearch-project:main Apr 4, 2022
@nknize nknize added backport 2.x Backport to 2.x branch backport 2.0 Backport to 2.0 branch labels Apr 4, 2022
nknize added a commit that referenced this pull request Apr 4, 2022
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 added a commit that referenced this pull request Apr 4, 2022
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 pushed a commit that referenced this pull request Apr 4, 2022
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 pushed a commit that referenced this pull request Apr 4, 2022
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)
@dblock
Copy link
Member

dblock commented Sep 7, 2022

This may have caused #3615, care to help @nknize?

@nknize
Copy link
Collaborator Author

nknize commented Sep 7, 2022

This may have caused #3615, care to help @nknize?

Ack, will take a look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Backport to 2.x branch backport 2.0 Backport to 2.0 branch non-issue bugs / unexpected behaviors that end up non issues; audit trail simple changes that aren't issues >upgrade Label used when upgrading library dependencies (e.g., Lucene) v3.0.0 Issues and PRs related to version 3.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants