Skip to content

Commit

Permalink
[CI] (Trivial) Update GitHub Actions
Browse files Browse the repository at this point in the history
Reviewers: anmolnar
Author: ctubbsii
Closes #2216 from ctubbsii/update-ga-v4

(cherry picked from commit aba99cf)
Signed-off-by: Andor Molnar <[email protected]>
  • Loading branch information
ctubbsii authored and anmolnar committed Nov 26, 2024
1 parent 43b97a7 commit de7e770
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 35 deletions.
19 changes: 6 additions & 13 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -48,20 +48,13 @@ jobs:
timeout-minutes: 360
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- name: Set up JDK ${{ matrix.profile.jdk }}
uses: actions/setup-java@v3
uses: actions/setup-java@v4
with:
java-version: ${{ matrix.profile.jdk }}
distribution: temurin
- name: Cache local maven repository
uses: actions/cache@v3
with:
path: |
~/.m2/repository/
!~/.m2/repository/org/apache/zookeeper
key: ${{ runner.os }}-m2-${{ hashFiles('**/pom.xml') }}
restore-keys: ${{ runner.os }}-m2
cache: 'maven'
- name: Show the first log message
run: git log -n1
- name: Install C Dependencies
Expand All @@ -74,21 +67,21 @@ jobs:
MAVEN_OPTS: -Djansi.force=true
- name: Upload unit test results
if: ${{ failure() }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: surefire-reports-${{ matrix.profile.name }}
path: ./**/target/surefire-reports/
if-no-files-found: ignore
- name: Upload integration test results
if: ${{ failure() }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: failsafe-reports-${{ matrix.profile.name }}
path: ./**/target/failsafe-reports/
if-no-files-found: ignore
- name: Upload cppunit test logs
if: ${{ failure() }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: cppunit-logs-${{ matrix.profile.name }}
path: ./zookeeper-client/zookeeper-client-c/target/c/TEST-*.txt
Expand Down
13 changes: 3 additions & 10 deletions .github/workflows/e2e.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,20 +31,13 @@ jobs:
timeout-minutes: 360
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- name: Set up JDK ${{ matrix.jdk }}
uses: actions/setup-java@v3
uses: actions/setup-java@v4
with:
java-version: ${{ matrix.jdk }}
distribution: temurin
- name: Cache local maven repository
uses: actions/cache@v3
with:
path: |
~/.m2/repository/
!~/.m2/repository/org/apache/zookeeper
key: ${{ runner.os }}-m2-${{ hashFiles('**/pom.xml') }}
restore-keys: ${{ runner.os }}-m2
cache: 'maven'
- name: Show the first log message
run: git log -n1
- name: Install C Dependencies
Expand Down
17 changes: 5 additions & 12 deletions .github/workflows/manual.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,22 +43,15 @@ jobs:
timeout-minutes: 360
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
ref: ${{ github.event.inputs.buildRef }}
- name: Set up JDK 11
uses: actions/setup-java@v3
uses: actions/setup-java@v4
with:
java-version: 11
distribution: temurin
- name: Cache local maven repository
uses: actions/cache@v3
with:
path: |
~/.m2/repository/
!~/.m2/repository/org/apache/zookeeper
key: ${{ runner.os }}-m2-${{ hashFiles('**/pom.xml') }}
restore-keys: ${{ runner.os }}-m2
cache: 'maven'
- name: Show the first log message
run: git log -n1
- name: Install C Dependencies
Expand All @@ -71,14 +64,14 @@ jobs:
MAVEN_OPTS: -Djansi.force=true
- name: Upload unit test results
if: ${{ failure() }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: surefire-reports
path: ./**/target/surefire-reports/
if-no-files-found: ignore
- name: Upload integration test results
if: ${{ failure() }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: failsafe-reports
path: ./**/target/failsafe-reports/
Expand Down

0 comments on commit de7e770

Please sign in to comment.