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

[#1097] enh(build,ci,jdk): run on jdk 11, 17, and 21 #1098

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
23 changes: 11 additions & 12 deletions .github/workflows/maven.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@ permissions:
on:
workflow_dispatch: {}
push:
branches: [ main, '1.11.x', '1.10.x', '1.9.x' ]
branches: [ main, '1.12.x', '1.11.x', '1.10.x' ]
pull_request:
branches: [ main, '1.11.x', '1.10.x', '1.9.x' ]
branches: [ main, '1.12.x', '1.11.x', '1.10.x' ]

jobs:
build:
Expand Down Expand Up @@ -55,23 +55,22 @@ jobs:
strategy:
matrix:
os: [ 'ubuntu-latest', 'windows-latest', 'macOS-latest' ]
jdk: [ 11, 17, 19 ]
jdk: [ 11, 17, 21 ]
dist: [ 'temurin', 'adopt-openj9', 'zulu' ]
exclude:
# was already built
- os: 'ubuntu-latest'
jdk: 11
dist: 'temurin'
# exclude some builds, because MacOs builds have fewer resources available.
# excludes java 16 on macOS.
- os: 'macOS-latest'
jdk: 16
# exclude temurin on MacOS. zulu (also hotspot) and OpenJ9 are sufficient.
- os: 'macOS-latest'
dist: 'temurin'
# no OpenJ9 19
# no Temurin 21
- dist: 'temurin'
jdk: 21
# no OpenJ9 21
- dist: adopt-openj9
jdk: 19
jdk: 21
fail-fast: false

runs-on: ${{ matrix.os }}
Expand All @@ -87,13 +86,13 @@ jobs:
distribution: ${{ matrix.dist }}

- name: Build with Maven (Linux)
if: matrix.os == 'ubuntu-latest'
if: matrix.os == 'ubuntu-latest' && matrix.jdk <= 17
run: >-
mvn verify --show-version --errors --batch-mode --no-transfer-progress -Pdocs
-Dgh_user=${{ github.actor }} -Dgh_token=${{ secrets.GITHUB_TOKEN }}

- name: Build with Maven (non-Linux)
if: matrix.os != 'ubuntu-latest'
- name: Build with Maven (non-Linux or JDK > 17)
if: matrix.os != 'ubuntu-latest' || matrix.jdk > 17
run: >-
mvn verify --show-version --errors --batch-mode --no-transfer-progress -Pdocs
-Pskip_jakarta_ee_tests
Expand Down
4 changes: 2 additions & 2 deletions .jenkins.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
* under the License.
*/

def deployableBranch = env.BRANCH_NAME ==~ /(1.7.x|1.8.x|1.9.x|main)/
def deployableBranch = env.BRANCH_NAME ==~ /(1.12.x|1.11.x|1.10.x|main)/

pipeline {

Expand All @@ -36,7 +36,7 @@ pipeline {
axis {
// https://cwiki.apache.org/confluence/display/INFRA/JDK+Installation+Matrix
name 'MATRIX_JDK'
values 'jdk_11_latest', 'jdk_17_latest', 'jdk_19_latest'
values 'jdk_11_latest', 'jdk_17_latest', 'jdk_21_latest'
}
// Additional axes, like OS and maven version can be configured here.
}
Expand Down