Skip to content

Commit 58e68eb

Browse files
authored
[ci] Print and upload logs for GitHub Action CI maven tests (#509)
1 parent 0626b16 commit 58e68eb

File tree

30 files changed

+380
-24
lines changed

30 files changed

+380
-24
lines changed

.github/workflows/ci.yaml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ jobs:
3737
- name: Test
3838
timeout-minutes: 60
3939
run: |
40-
mvn -B verify -Ptest-coverage
40+
mvn -B verify -Ptest-coverage -Dlog.dir=${{ runner.temp }}/fluss-logs -Dlog4j.configurationFile=${{ github.workspace }}/tools/ci/log4j.properties
4141
env:
4242
MAVEN_OPTS: -Xmx4096m
4343
ARTIFACTS_OSS_ENDPOINT: ${{ secrets.ARTIFACTS_OSS_ENDPOINT }}
@@ -46,4 +46,10 @@ jobs:
4646
ARTIFACTS_OSS_ACCESS_KEY: ${{ secrets.ARTIFACTS_OSS_ACCESS_KEY }}
4747
ARTIFACTS_OSS_SECRET_KEY: ${{ secrets.ARTIFACTS_OSS_SECRET_KEY }}
4848
ARTIFACTS_OSS_STS_ENDPOINT: ${{ secrets.ARTIFACTS_OSS_STS_ENDPOINT }}
49-
ARTIFACTS_OSS_ROLE_ARN: ${{ secrets.ARTIFACTS_OSS_ROLE_ARN }}
49+
ARTIFACTS_OSS_ROLE_ARN: ${{ secrets.ARTIFACTS_OSS_ROLE_ARN }}
50+
- name: Upload build logs
51+
uses: actions/upload-artifact@v4
52+
if: ${{ failure() }}
53+
with:
54+
name: logs-test-${{ github.run_number}}#${{ github.run_attempt }}
55+
path: ${{ runner.temp }}/fluss-logs/*

fluss-client/pom.xml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,8 +81,6 @@
8181
<dependency>
8282
<groupId>com.alibaba.fluss</groupId>
8383
<artifactId>fluss-test-utils</artifactId>
84-
<version>${project.version}</version>
85-
<scope>test</scope>
8684
</dependency>
8785

8886
<dependency>
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
#
2+
# Copyright (c) 2025 Alibaba Group Holding Ltd.
3+
#
4+
# Licensed under the Apache License, Version 2.0 (the "License");
5+
# you may not use this file except in compliance with the License.
6+
# You may obtain a copy of the License at
7+
#
8+
# http://www.apache.org/licenses/LICENSE-2.0
9+
#
10+
# Unless required by applicable law or agreed to in writing, software
11+
# distributed under the License is distributed on an "AS IS" BASIS,
12+
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
# See the License for the specific language governing permissions and
14+
# limitations under the License.
15+
#
16+
17+
com.alibaba.fluss.testutils.common.TestLoggerExtension

fluss-common/pom.xml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,8 +86,6 @@
8686
<dependency>
8787
<groupId>com.alibaba.fluss</groupId>
8888
<artifactId>fluss-test-utils</artifactId>
89-
<version>${project.version}</version>
90-
<scope>test</scope>
9189
</dependency>
9290
<!-- test util for ProtoCodecUtilsTest -->
9391
<dependency>

fluss-common/src/test/resources/resources/META-INF/services/com.alibaba.fluss.fs.FileSystemPlugin renamed to fluss-common/src/test/resources/META-INF/services/com.alibaba.fluss.fs.FileSystemPlugin

File renamed without changes.
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
#
2+
# Copyright (c) 2025 Alibaba Group Holding Ltd.
3+
#
4+
# Licensed under the Apache License, Version 2.0 (the "License");
5+
# you may not use this file except in compliance with the License.
6+
# You may obtain a copy of the License at
7+
#
8+
# http://www.apache.org/licenses/LICENSE-2.0
9+
#
10+
# Unless required by applicable law or agreed to in writing, software
11+
# distributed under the License is distributed on an "AS IS" BASIS,
12+
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
# See the License for the specific language governing permissions and
14+
# limitations under the License.
15+
#
16+
17+
com.alibaba.fluss.testutils.common.TestLoggerExtension

fluss-connectors/fluss-connector-flink/pom.xml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -143,8 +143,6 @@
143143
<dependency>
144144
<groupId>com.alibaba.fluss</groupId>
145145
<artifactId>fluss-test-utils</artifactId>
146-
<version>${project.version}</version>
147-
<scope>test</scope>
148146
</dependency>
149147

150148
<dependency>
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
#
2+
# Copyright (c) 2025 Alibaba Group Holding Ltd.
3+
#
4+
# Licensed under the Apache License, Version 2.0 (the "License");
5+
# you may not use this file except in compliance with the License.
6+
# You may obtain a copy of the License at
7+
#
8+
# http://www.apache.org/licenses/LICENSE-2.0
9+
#
10+
# Unless required by applicable law or agreed to in writing, software
11+
# distributed under the License is distributed on an "AS IS" BASIS,
12+
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
# See the License for the specific language governing permissions and
14+
# limitations under the License.
15+
#
16+
17+
com.alibaba.fluss.testutils.common.TestLoggerExtension

fluss-filesystems/fluss-fs-hadoop/pom.xml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,8 +88,6 @@
8888
<dependency>
8989
<groupId>com.alibaba.fluss</groupId>
9090
<artifactId>fluss-test-utils</artifactId>
91-
<version>${project.version}</version>
92-
<scope>test</scope>
9391
</dependency>
9492

9593
<!-- for the HDFS mini cluster test suite -->
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
#
2+
# Copyright (c) 2025 Alibaba Group Holding Ltd.
3+
#
4+
# Licensed under the Apache License, Version 2.0 (the "License");
5+
# you may not use this file except in compliance with the License.
6+
# You may obtain a copy of the License at
7+
#
8+
# http://www.apache.org/licenses/LICENSE-2.0
9+
#
10+
# Unless required by applicable law or agreed to in writing, software
11+
# distributed under the License is distributed on an "AS IS" BASIS,
12+
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
# See the License for the specific language governing permissions and
14+
# limitations under the License.
15+
#
16+
17+
com.alibaba.fluss.testutils.common.TestLoggerExtension

0 commit comments

Comments
 (0)