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.

Sign up for GitHub

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

Jump to bottom

Default CI Java Version to Java 11, run tests on 8, 14 and 17 #254

Closed
3 tasks
Tracked by #64
dblock opened this issue Nov 17, 2021 · 2 comments
Closed
3 tasks
Tracked by #64

Default CI Java Version to Java 11, run tests on 8, 14 and 17 #254

dblock opened this issue Nov 17, 2021 · 2 comments
Labels
untriaged v1.3.0

Comments

@dblock
Copy link
Member

dblock commented Nov 17, 2021 •
edited by naveenpajjuri
Loading

Coming from opensearch-project/opensearch-plugins#64, default CI Java version to 11 (LTS), and run tests on 8, 14 and 17.

  • Change the default Java version for CI in GitHub actions from 14/15 to 11 (LTS).
  • Add Java 8, 14 (used before), and 17 (next LTS) to the CI matrix.
  • Document Java versions in DEVELOPER_GUIDE.md.
@dblock dblock mentioned this issue Nov 17, 2021
Closed
15 tasks
@naveenpajjuri naveenpajjuri mentioned this issue Mar 1, 2022
Closed
naveenpajjuri pushed a commit that referenced this issue Mar 2, 2022
changed default java version (#254)
9a7c555 
Signed-off-by: Naveen Pajjuri <nppajjur@amazon.com>
@naveenpajjuri naveenpajjuri mentioned this issue Mar 2, 2022
Merged
5 tasks
naveenpajjuri pushed a commit that referenced this issue Mar 8, 2022
@naveenpajjuri
changed default java version (#254)
c068e26 
Signed-off-by: Naveen Pajjuri <nppajjur@amazon.com>
@naveenpajjuri naveenpajjuri mentioned this issue Mar 8, 2022
Merged
5 tasks
@dblock
Copy link
Member Author

dblock commented Mar 9, 2022

@naveenpajjuri what else is left to do to close this for 1.3.0?

@naveenpajjuri
Copy link
Contributor

naveenpajjuri commented Mar 10, 2022 •
edited
Loading

Currently we have CI support for 8,11 and 14. with default version as 11.
We need to support JDK 17 as well, for which we need to upgrade gradle version to 7, which currently needs work (#328).

@dblock : closing this as i see you have already opened an issue for JDK 17 support (#331)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
untriaged v1.3.0
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dblock @naveenpajjuri