Skip to content
This repository has been archived by the owner on Apr 1, 2024. It is now read-only.

Commit

Permalink
[improve][ci] Switch JDK distribution from temurin to corretto (apach…
Browse files Browse the repository at this point in the history
  • Loading branch information
nodece authored Mar 11, 2024
1 parent b4f710d commit d61f758
Show file tree
Hide file tree
Showing 5 changed files with 25 additions and 18 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/ci-maven-cache-update.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ on:

env:
MAVEN_OPTS: -Xss1500k -Xmx1024m -Daether.connector.http.reuseConnections=false -Daether.connector.requestTimeout=60000 -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false -Dmaven.wagon.http.retryHandler.class=standard -Dmaven.wagon.http.retryHandler.count=3 -Dmaven.wagon.http.retryHandler.requestSentEnabled=true -Dmaven.wagon.http.serviceUnavailableRetryStrategy.class=standard -Dmaven.wagon.rto=60000
JDK_DISTRIBUTION: corretto

jobs:
update-maven-dependencies-cache:
Expand Down Expand Up @@ -106,7 +107,7 @@ jobs:
uses: actions/setup-java@v4
if: ${{ (github.event_name == 'schedule' || steps.changes.outputs.poms == 'true') && steps.cache.outputs.cache-hit != 'true' }}
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: 17

- name: Download dependencies
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/ci-owasp-dependency-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ on:

env:
MAVEN_OPTS: -Xss1500k -Xmx1024m -Daether.connector.http.reuseConnections=false -Daether.connector.requestTimeout=60000 -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false -Dmaven.wagon.http.retryHandler.class=standard -Dmaven.wagon.http.retryHandler.count=3 -Dmaven.wagon.http.retryHandler.requestSentEnabled=true -Dmaven.wagon.http.serviceUnavailableRetryStrategy.class=standard -Dmaven.wagon.rto=60000
JDK_DISTRIBUTION: corretto

jobs:
run-owasp-dependency-check:
Expand Down Expand Up @@ -74,7 +75,7 @@ jobs:
- name: Set up JDK ${{ matrix.jdk || '17' }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ matrix.jdk || '17' }}

- name: run install by skip tests
Expand Down
5 changes: 4 additions & 1 deletion .github/workflows/codeql.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,9 @@ concurrency:
group: ${{ github.workflow }}-${{ github.ref }}-${{ github.event_name }}
cancel-in-progress: true

env:
JDK_DISTRIBUTION: corretto

jobs:
analyze:
# only run scheduled analysis in apache/pulsar repository
Expand Down Expand Up @@ -63,7 +66,7 @@ jobs:
- name: Set up JDK
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: 17

- name: Checkout repository
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/pulsar-ci-flaky.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,7 @@ env:
# it's possible to rerun individual failed jobs when the build artifacts are available
# if the artifacts have already been expired, the complete workflow can be rerun by closing and reopening the PR or by rebasing the PR
ARTIFACT_RETENTION_DAYS: 3
JDK_DISTRIBUTION: corretto

jobs:
preconditions:
Expand Down Expand Up @@ -189,7 +190,7 @@ jobs:
- name: Set up JDK ${{ env.CI_JDK_MAJOR_VERSION }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ env.CI_JDK_MAJOR_VERSION }}

- name: Build core-modules
Expand Down
29 changes: 15 additions & 14 deletions .github/workflows/pulsar-ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,7 @@ env:
# it's possible to rerun individual failed jobs when the build artifacts are available
# if the artifacts have already been expired, the complete workflow can be rerun by closing and reopening the PR or by rebasing the PR
ARTIFACT_RETENTION_DAYS: 3
JDK_DISTRIBUTION: corretto

jobs:
preconditions:
Expand Down Expand Up @@ -180,7 +181,7 @@ jobs:
- name: Set up JDK ${{ env.CI_JDK_MAJOR_VERSION }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ env.CI_JDK_MAJOR_VERSION }}

- name: Check source code license headers
Expand Down Expand Up @@ -299,7 +300,7 @@ jobs:
- name: Set up JDK ${{ matrix.jdk || env.CI_JDK_MAJOR_VERSION }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ matrix.jdk || env.CI_JDK_MAJOR_VERSION }}

- name: Install gh-actions-artifact-client.js
Expand Down Expand Up @@ -415,7 +416,7 @@ jobs:
- name: Set up JDK ${{ matrix.jdk || env.CI_JDK_MAJOR_VERSION }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ matrix.jdk || env.CI_JDK_MAJOR_VERSION }}

- name: Install gh-actions-artifact-client.js
Expand Down Expand Up @@ -497,7 +498,7 @@ jobs:
- name: Set up JDK ${{ env.CI_JDK_MAJOR_VERSION }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ env.CI_JDK_MAJOR_VERSION }}

- name: Install gh-actions-artifact-client.js
Expand Down Expand Up @@ -618,7 +619,7 @@ jobs:
- name: Set up JDK ${{ env.CI_JDK_MAJOR_VERSION }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ env.CI_JDK_MAJOR_VERSION }}

- name: Install gh-actions-artifact-client.js
Expand All @@ -643,7 +644,7 @@ jobs:
uses: actions/setup-java@v4
if: ${{ matrix.runtime_jdk }}
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ matrix.runtime_jdk }}

- name: Run integration test group '${{ matrix.group }}'
Expand Down Expand Up @@ -735,7 +736,7 @@ jobs:
- name: Set up JDK ${{ env.CI_JDK_MAJOR_VERSION }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ env.CI_JDK_MAJOR_VERSION }}

- name: Install gh-actions-artifact-client.js
Expand Down Expand Up @@ -852,7 +853,7 @@ jobs:
- name: Set up JDK ${{ env.CI_JDK_MAJOR_VERSION }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ env.CI_JDK_MAJOR_VERSION }}

- name: Install gh-actions-artifact-client.js
Expand Down Expand Up @@ -982,7 +983,7 @@ jobs:
- name: Set up JDK ${{ env.CI_JDK_MAJOR_VERSION }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ env.CI_JDK_MAJOR_VERSION }}

- name: Install gh-actions-artifact-client.js
Expand Down Expand Up @@ -1094,7 +1095,7 @@ jobs:
- name: Set up JDK ${{ env.CI_JDK_MAJOR_VERSION }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ env.CI_JDK_MAJOR_VERSION }}

- name: Install gh-actions-artifact-client.js
Expand Down Expand Up @@ -1197,7 +1198,7 @@ jobs:
- name: Set up JDK ${{ env.CI_JDK_MAJOR_VERSION }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ env.CI_JDK_MAJOR_VERSION }}

- name: Install gh-actions-artifact-client.js
Expand Down Expand Up @@ -1316,7 +1317,7 @@ jobs:
- name: Set up JDK ${{ env.CI_JDK_MAJOR_VERSION }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ env.CI_JDK_MAJOR_VERSION }}

- name: build package
Expand Down Expand Up @@ -1370,7 +1371,7 @@ jobs:
- name: Set up JDK ${{ env.CI_JDK_MAJOR_VERSION }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ env.CI_JDK_MAJOR_VERSION }}

- name: Initialize CodeQL
Expand Down Expand Up @@ -1427,7 +1428,7 @@ jobs:
- name: Set up JDK ${{ matrix.jdk || env.CI_JDK_MAJOR_VERSION }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
distribution: ${{ env.JDK_DISTRIBUTION }}
java-version: ${{ matrix.jdk || env.CI_JDK_MAJOR_VERSION }}

- name: Clean Disk
Expand Down

0 comments on commit d61f758

Please sign in to comment.