This is an automated email from the ASF dual-hosted git repository.
tison pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/zookeeper.git
The following commit(s) were added to refs/heads/master by this push:
new 14b936201 ZOOKEEPER-4679: Fix CI failure and warning (#1987)
14b936201 is described below
commit 14b936201b719ab5cf6346d1ed7dfc4ff382cbb7
Author: tison <[email protected]>
AuthorDate: Mon Mar 6 15:57:47 2023 +0800
ZOOKEEPER-4679: Fix CI failure and warning (#1987)
1. Run `apt update` before installing C deps to avoid 404.
2. Update actions version to avoid set-output deprecation warnings.
Signed-off-by: tison <[email protected]>
---
.github/workflows/ci.yaml | 16 ++++++++--------
.github/workflows/e2e.yaml | 9 ++++++---
.github/workflows/manual.yaml | 16 ++++++++--------
3 files changed, 22 insertions(+), 19 deletions(-)
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
index 45ff90cc6..cafc94729 100644
--- a/.github/workflows/ci.yaml
+++ b/.github/workflows/ci.yaml
@@ -1,4 +1,3 @@
-#
# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements. See the NOTICE file
# distributed with this work for additional information
@@ -15,7 +14,6 @@
# KIND, either express or implied. See the License for the
# specific language governing permissions and limitations
# under the License.
-#
# This workflow will build a Java project with Maven
# See also:
@@ -52,11 +50,12 @@ jobs:
steps:
- uses: actions/checkout@v3
- name: Set up JDK ${{ matrix.profile.jdk }}
- uses: actions/setup-java@v1
+ uses: actions/setup-java@v3
with:
java-version: ${{ matrix.profile.jdk }}
+ distribution: temurin
- name: Cache local maven repository
- uses: actions/cache@v2
+ uses: actions/cache@v3
with:
path: |
~/.m2/repository/
@@ -66,23 +65,24 @@ jobs:
- name: Show the first log message
run: git log -n1
- name: Install C Dependencies
- run: sudo apt-get install libcppunit-dev libsasl2-dev
+ run: |
+ sudo apt update
+ sudo apt install -y libcppunit-dev libsasl2-dev
- name: Build with Maven (${{ matrix.profile.name }})
run: mvn -B -V -e -ntp "-Dstyle.color=always" ${{ matrix.profile.args }}
env:
MAVEN_OPTS: -Djansi.force=true
- name: Upload unit test results
if: ${{ failure() }}
- uses: actions/upload-artifact@v2
+ uses: actions/upload-artifact@v3
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@v2
+ uses: actions/upload-artifact@v3
with:
name: failsafe-reports-${{ matrix.profile.name }}
path: ./**/target/failsafe-reports/
if-no-files-found: ignore
-
diff --git a/.github/workflows/e2e.yaml b/.github/workflows/e2e.yaml
index ea3c85e98..e497c3f03 100644
--- a/.github/workflows/e2e.yaml
+++ b/.github/workflows/e2e.yaml
@@ -33,11 +33,12 @@ jobs:
steps:
- uses: actions/checkout@v3
- name: Set up JDK ${{ matrix.jdk }}
- uses: actions/setup-java@v1
+ uses: actions/setup-java@v3
with:
java-version: ${{ matrix.jdk }}
+ distribution: temurin
- name: Cache local maven repository
- uses: actions/cache@v2
+ uses: actions/cache@v3
with:
path: |
~/.m2/repository/
@@ -47,7 +48,9 @@ jobs:
- name: Show the first log message
run: git log -n1
- name: Install C Dependencies
- run: sudo apt-get install libcppunit-dev libsasl2-dev
+ run: |
+ sudo apt update
+ sudo apt install -y libcppunit-dev libsasl2-dev
- name: Build with Maven
run: mvn -B -V -e -ntp "-Dstyle.color=always" package -DskipTests
env:
diff --git a/.github/workflows/manual.yaml b/.github/workflows/manual.yaml
index ca00519c9..905ba49f3 100644
--- a/.github/workflows/manual.yaml
+++ b/.github/workflows/manual.yaml
@@ -1,4 +1,3 @@
-#
# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements. See the NOTICE file
# distributed with this work for additional information
@@ -15,7 +14,6 @@
# KIND, either express or implied. See the License for the
# specific language governing permissions and limitations
# under the License.
-#
# This workflow will build a Java project with Maven
# See also:
@@ -49,11 +47,12 @@ jobs:
with:
ref: ${{ github.event.inputs.buildRef }}
- name: Set up JDK 11
- uses: actions/setup-java@v1
+ uses: actions/setup-java@v3
with:
java-version: 11
+ distribution: temurin
- name: Cache local maven repository
- uses: actions/cache@v2
+ uses: actions/cache@v3
with:
path: |
~/.m2/repository/
@@ -63,23 +62,24 @@ jobs:
- name: Show the first log message
run: git log -n1
- name: Install C Dependencies
- run: sudo apt-get install libcppunit-dev libsasl2-dev
+ run: |
+ sudo apt update
+ sudo apt install -y libcppunit-dev libsasl2-dev
- name: Build with Maven
run: mvn -B -V -e -ntp "-Dstyle.color=always" ${{
github.event.inputs.mvnOpts }} ${{ github.event.inputs.goals }}
env:
MAVEN_OPTS: -Djansi.force=true
- name: Upload unit test results
if: ${{ failure() }}
- uses: actions/upload-artifact@v2
+ uses: actions/upload-artifact@v3
with:
name: surefire-reports
path: ./**/target/surefire-reports/
if-no-files-found: ignore
- name: Upload integration test results
if: ${{ failure() }}
- uses: actions/upload-artifact@v2
+ uses: actions/upload-artifact@v3
with:
name: failsafe-reports
path: ./**/target/failsafe-reports/
if-no-files-found: ignore
-
