Skip to content

Commit

Permalink
[build] Fix JDK8 builds and set up CI (#627)
Browse files Browse the repository at this point in the history
Previously, builds using JDK8 would fail. Also, cleaned up the alpn-boot loading process for HTTP/2 on JDK 8 tests.
  • Loading branch information
nisargthakkar authored Sep 9, 2023
1 parent 1c13637 commit 5b5feaa
Show file tree
Hide file tree
Showing 6 changed files with 89 additions and 97 deletions.
4 changes: 2 additions & 2 deletions .github/rawWorkflows/gh-ci-alpini-parametrized-flow.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
strategy:
fail-fast: false
matrix:
jdk: [11, 17]
jdk: [8, 11, 17]
runs-on: ubuntu-latest
timeout-minutes: $TimeOut
outputs:
Expand All @@ -28,7 +28,7 @@
if: steps.check_alpini_files_changed.outputs.alpini == 'true'
with:
java-version: ${{ matrix.jdk }}
distribution: 'microsoft'
distribution: 'temurin'
cache: 'gradle'
# - name: Allow Deprecated TLS versions for Alpini tests
# run: |
Expand Down
4 changes: 2 additions & 2 deletions .github/rawWorkflows/gh-ci-parameterized-flow.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
strategy:
fail-fast: false
matrix:
jdk: [11, 17]
jdk: [8, 11, 17]
runs-on: ubuntu-latest
needs: $Dependency
if: $Conditional
Expand All @@ -15,7 +15,7 @@
uses: actions/setup-java@v3
with:
java-version: ${{ matrix.jdk }}
distribution: 'microsoft'
distribution: 'temurin'
cache: 'gradle'
- shell: bash
run: |
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/gh-ci-pulsar-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ jobs:
strategy:
fail-fast: false
matrix:
jdk: [11, 17]
jdk: [8, 11, 17]
runs-on: ubuntu-latest
timeout-minutes: 120
steps:
Expand All @@ -19,7 +19,7 @@ jobs:
uses: actions/setup-java@v3
with:
java-version: ${{ matrix.jdk }}
distribution: 'microsoft'
distribution: 'temurin'
cache: 'gradle'
- shell: bash
run: |
Expand Down
Loading

0 comments on commit 5b5feaa

Please sign in to comment.