Skip to content

Commit

Permalink
Merge pull request #1098 from lprimak/1097-enhancement-update-ci-to-u…
Browse files Browse the repository at this point in the history
…se-jdk-11-17-and-21

[#1097] enh(build,ci,jdk): run on jdk 11, 17, and 21
  • Loading branch information
lprimak committed Oct 1, 2023
2 parents 938831d + 888d5a1 commit 5b02303
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 14 deletions.
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

0 comments on commit 5b02303

Please sign in to comment.