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

Fix build-tools/reaper source/target compatibility to be JDK-11 #2596

Merged
merged 1 commit into from
Mar 25, 2022

Conversation

reta
Copy link
Collaborator

@reta reta commented Mar 25, 2022

Signed-off-by: Andriy Redko [email protected]

Description

Fix build-tools/reaper source/target compatibility to be JDK-11

Issues Resolved

Fixes #2581

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@reta reta requested a review from a team as a code owner March 25, 2022 16:31
@reta
Copy link
Collaborator Author

reta commented Mar 25, 2022

@dblock please

@Rishikesh1159 Rishikesh1159 requested a review from dblock March 25, 2022 16:40
@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure ce02a4980878154539bbf079867a31876055964e
Log 3758

Reports 3758

@reta
Copy link
Collaborator Author

reta commented Mar 25, 2022

x Gradle Check failure ce02a49 Log 3758

Reports 3758

#1561 :(

Copy link
Member

@dblock dblock left a comment

Choose a reason for hiding this comment

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

@Rishikesh1159

  1. please audit all the JARs that we output and fix any others that are not source/target = 11
  2. open an issue to 1) find a way to inherit source/target compatibility from 1 place 2) find a systematic way to fail the build when some distribution JARs aren't the same version

@dblock
Copy link
Member

dblock commented Mar 25, 2022

@reta Thanks for jumping in.

@reta
Copy link
Collaborator Author

reta commented Mar 25, 2022

  1. please audit all the JARs that we output and fix any others that are not source/target = 11

We have another outlier in distribution/tools/java-version-checker/build.gradle

@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure fe1d4f24d35aa7d82c5d78bdbea37006a559d10a
Log 3759

Reports 3759

@reta
Copy link
Collaborator Author

reta commented Mar 25, 2022

x Gradle Check failure fe1d4f2 Log 3759

Reports 3759

#1703 :(

@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure 34864f14ec00c86b805b9e3dbeebe21e84fced91
Log 3760

Reports 3760

@reta
Copy link
Collaborator Author

reta commented Mar 25, 2022

#1828 :(

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success 426205b
Log 3763

Reports 3763

@dblock dblock merged commit d4ce87b into opensearch-project:main Mar 25, 2022
@dblock dblock added backport 2.x Backport to 2.x branch backport 2.0 Backport to 2.0 branch labels Mar 25, 2022
opensearch-trigger-bot bot pushed a commit that referenced this pull request Mar 25, 2022
opensearch-trigger-bot bot pushed a commit that referenced this pull request Mar 25, 2022
peterzhuamazon pushed a commit that referenced this pull request Mar 25, 2022
… (#2605)

Signed-off-by: Andriy Redko <[email protected]>
(cherry picked from commit d4ce87b)

Co-authored-by: Andriy Redko <[email protected]>
peterzhuamazon pushed a commit that referenced this pull request Mar 25, 2022
… (#2606)

Signed-off-by: Andriy Redko <[email protected]>
(cherry picked from commit d4ce87b)

Co-authored-by: Andriy Redko <[email protected]>
@vmmusings
Copy link
Member

I am facing this issue when I am building 1.2 version. Should this be back ported to 1.x branches with 1.8 Java Version?
@dblock ?

@reta
Copy link
Collaborator Author

reta commented Aug 30, 2022

I am facing this issue when I am building 1.2 version. Should this be back ported to 1.x branches with 1.8 Java Version? @dblock ?

@vamsi-amazon the 1.x has JDK 1.8 baseline, this change should not be backported

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug][DISTRIBUTION][2.0.0]: Plugin build fails for JDK lower than 17 (11 and 14)
5 participants