Skip to content

Commit

Permalink
[CELEBORN-1556] Update Github actions to v4
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?

- update actions/setup-java to v4
- update actions/upload-artifact to v4
- update actions/checkout to v4
- update pnpm/action-setup to v4

### Why are the changes needed?

Mordernize the github actions used, bumping the runtime in actions to NodeJS 20.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Passing the CI jobs.

Closes #2676 from bowenliang123/update-actions.

Lead-authored-by: Bowen Liang <[email protected]>
Co-authored-by: liangbowen <[email protected]>
Signed-off-by: SteNicholas <[email protected]>
  • Loading branch information
bowenliang123 authored and SteNicholas committed Aug 16, 2024
1 parent 2ad44ba commit 0ee3c3a
Show file tree
Hide file tree
Showing 10 changed files with 49 additions and 49 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/benchmark.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ jobs:
runs-on: self-hosted
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
repository: ${{ github.event.client_payload.pull_request.head.repo.full_name }}
ref: ${{ github.event.client_payload.pull_request.head.ref }}
Expand All @@ -42,7 +42,7 @@ jobs:
run: /home/hadoop/celeborn-toolkit/reg.sh benchmark

- name: Upload Result
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: benchmark result
path: /home/hadoop/celeborn-toolkit/result/
4 changes: 2 additions & 2 deletions .github/workflows/benchmark_manual.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ jobs:
runs-on: self-hosted
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Build
run: |
Expand All @@ -42,7 +42,7 @@ jobs:
run: /home/hadoop/celeborn-toolkit/reg.sh benchmark

- name: Upload Result
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: benchmark result
path: /home/hadoop/celeborn-toolkit/result/
8 changes: 4 additions & 4 deletions .github/workflows/deps.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,9 +57,9 @@ jobs:
- 'flink-1.20'
- 'mr'
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup JDK 8
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
distribution: zulu
java-version: 8
Expand Down Expand Up @@ -91,9 +91,9 @@ jobs:
- 'flink-1.20'
- 'mr'
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup JDK 8
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
distribution: zulu
java-version: 8
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/license.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,9 @@ jobs:
name: License
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup JDK 8
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
distribution: zulu
java-version: 8
Expand All @@ -54,7 +54,7 @@ jobs:
build/mvn org.apache.rat:apache-rat-plugin:check -Pgoogle-mirror,mr
- name: Upload rat report
if: failure()
uses: actions/upload-artifact@v2
uses: actions/upload-artifact@v4
with:
name: rat-report
path: "**/target/rat*.txt"
30 changes: 15 additions & 15 deletions .github/workflows/maven.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,9 @@ jobs:
- 11
- 17
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup JDK ${{ matrix.java }}
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
distribution: zulu
java-version: ${{ matrix.java }}
Expand All @@ -36,7 +36,7 @@ jobs:
run: build/mvn -Pgoogle-mirror test
- name: Upload test log
if: failure()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: service-unit-test-log
path: |
Expand All @@ -55,9 +55,9 @@ jobs:
spark:
- '2.4'
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup JDK ${{ matrix.java }}
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
distribution: zulu
java-version: ${{ matrix.java }}
Expand All @@ -73,7 +73,7 @@ jobs:
build/mvn $PROFILES -pl $TEST_MODULES test
- name: Upload test log
if: failure()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: spark-${{ matrix.spark }}-unit-test-log
path: |
Expand Down Expand Up @@ -118,9 +118,9 @@ jobs:
- shuffle-plugin-class: 'org.apache.spark.shuffle.celeborn.CelebornShuffleDataIO'
spark: '3.4'
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup JDK ${{ matrix.java }}
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
distribution: zulu
java-version: ${{ matrix.java }}
Expand All @@ -136,7 +136,7 @@ jobs:
build/mvn $PROFILES -pl $TEST_MODULES -Dspark.shuffle.sort.io.plugin.class=${{ matrix.shuffle-plugin-class }} test
- name: Upload test log
if: failure()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: spark-${{ matrix.spark }}-unit-test-log
path: |
Expand All @@ -159,9 +159,9 @@ jobs:
- '1.19'
- '1.20'
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup JDK ${{ matrix.java }}
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
distribution: zulu
java-version: ${{ matrix.java }}
Expand All @@ -175,7 +175,7 @@ jobs:
build/mvn $PROFILES -pl $TEST_MODULES test
- name: Upload test log
if: failure()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: flink-${{ matrix.flink }}-unit-test-log
path: |
Expand All @@ -190,9 +190,9 @@ jobs:
- 8
- 11
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup JDK ${{ matrix.java }}
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
distribution: zulu
java-version: ${{ matrix.java }}
Expand All @@ -206,7 +206,7 @@ jobs:
build/mvn $PROFILES -pl $TEST_MODULES test
- name: Upload test log
if: failure()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: mr-unit-test-log
path: |
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/regression.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ jobs:
runs-on: self-hosted
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
repository: ${{ github.event.client_payload.pull_request.head.repo.full_name }}
ref: ${{ github.event.client_payload.pull_request.head.ref }}
Expand All @@ -42,7 +42,7 @@ jobs:
run: /home/hadoop/celeborn-toolkit/reg.sh regression

- name: Upload Result
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: regression result
path: /home/hadoop/celeborn-toolkit/result/
4 changes: 2 additions & 2 deletions .github/workflows/regression_manual.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ jobs:
runs-on: self-hosted
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Build
run: |
Expand All @@ -43,7 +43,7 @@ jobs:
run: /home/hadoop/celeborn-toolkit/reg.sh regression

- name: Upload Result
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: regression result
path: /home/hadoop/celeborn-toolkit/result/
30 changes: 15 additions & 15 deletions .github/workflows/sbt.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,9 +42,9 @@ jobs:
- '2.12.15'
- '2.13.5'
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup JDK ${{ matrix.java }}
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
distribution: zulu
java-version: ${{ matrix.java }}
Expand All @@ -54,7 +54,7 @@ jobs:
build/sbt ++${{ matrix.scala }} "clean; test"
- name: Upload test log
if: failure()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: service-java-${{ matrix.java }}-scala-${{ matrix.scala }}-unit-test-log
path: |
Expand All @@ -73,9 +73,9 @@ jobs:
- '2.11.12'
- '2.12.10'
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup JDK ${{ matrix.java }}
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
distribution: zulu
java-version: ${{ matrix.java }}
Expand All @@ -85,7 +85,7 @@ jobs:
build/sbt -Pspark-${{ matrix.spark }} ++${{ matrix.scala }} "clean; celeborn-spark-group/test"
- name: Upload test log
if: failure()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: spark-${{ matrix.spark }}-unit-test-log
path: |
Expand Down Expand Up @@ -176,9 +176,9 @@ jobs:
scala-binary: '2.13'
scala: '2.13.5'
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup JDK ${{ matrix.java }}
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
distribution: zulu
java-version: ${{ matrix.java }}
Expand All @@ -188,7 +188,7 @@ jobs:
build/sbt -Dspark.shuffle.plugin.class=${{ matrix.shuffle-plugin-class }} -Pspark-${{ matrix.spark }} ++${{ matrix.scala }} "clean; celeborn-spark-group/test"
- name: Upload test log
if: failure()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: spark-${{ matrix.spark }}-scala-${{ matrix.scala }}-unit-test-log
path: |
Expand All @@ -211,9 +211,9 @@ jobs:
- '1.19'
- '1.20'
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup JDK ${{ matrix.java }}
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
distribution: zulu
java-version: ${{ matrix.java }}
Expand All @@ -223,7 +223,7 @@ jobs:
build/sbt -Pflink-${{ matrix.flink }} "clean; celeborn-flink-group/test"
- name: Upload test log
if: failure()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: flink-${{ matrix.flink }}-unit-test-log
path: |
Expand All @@ -238,9 +238,9 @@ jobs:
- 8
- 11
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup JDK ${{ matrix.java }}
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
distribution: zulu
java-version: ${{ matrix.java }}
Expand All @@ -250,7 +250,7 @@ jobs:
build/sbt -Pmr "clean; celeborn-mr-group/test"
- name: Upload test log
if: failure()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: mr-unit-test-log
path: |
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/style.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,9 @@ jobs:
strategy:
fail-fast: false
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup JDK 8
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
distribution: zulu
java-version: 8
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/web_lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ jobs:
runs-on: ubuntu-latest

steps:
- uses: pnpm/action-setup@v2
- uses: pnpm/action-setup@v4
with:
version: 8

Expand All @@ -50,7 +50,7 @@ jobs:
node-version: 20.11.0

- name: Use pnpm v8.14.3
uses: pnpm/action-setup@v2
uses: pnpm/action-setup@v4
with:
version: 8.14.3

Expand Down

0 comments on commit 0ee3c3a

Please sign in to comment.