Skip to content

Commit b189591

Browse files
committed
[KYUUBI #2754] [GA] Separate log archive name
### _Why are the changes needed?_ Currently the GA jobs share the name log archive file name, it may overwrite each other if multi jobs fail. ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible - [ ] Add screenshots for manual tests if appropriate - [ ] [Run test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests) locally before make a pull request Closes #2754 from pan3793/ga-log. Closes #2754 d3b7469 [Cheng Pan] [GA] Seperate log archive name Authored-by: Cheng Pan <chengpan@apache.org> Signed-off-by: Cheng Pan <chengpan@apache.org>
1 parent 1ca56c1 commit b189591

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

.github/workflows/master.yml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,19 +49,23 @@ jobs:
4949
- '3.3'
5050
spark-archive: [""]
5151
exclude-tags: [""]
52+
comment: ["normal"]
5253
include:
5354
- java: 8
5455
spark: '3.2'
5556
spark-archive: '-Dspark.archive.mirror=https://archive.apache.org/dist/spark/spark-3.0.3 -Dspark.archive.name=spark-3.0.3-bin-hadoop2.7.tgz'
5657
exclude-tags: '-Dmaven.plugin.scalatest.exclude.tags=org.apache.kyuubi.tags.ExtendedSQLTest,org.apache.kyuubi.tags.DeltaTest,org.apache.kyuubi.tags.HudiTest,org.apache.kyuubi.tags.IcebergTest'
58+
comment: 'verify-spark-3.0'
5759
- java: 8
5860
spark: '3.2'
5961
spark-archive: '-Dspark.archive.mirror=https://archive.apache.org/dist/spark/spark-3.1.3 -Dspark.archive.name=spark-3.1.3-bin-hadoop3.2.tgz'
6062
exclude-tags: '-Dmaven.plugin.scalatest.exclude.tags=org.apache.kyuubi.tags.ExtendedSQLTest,org.apache.kyuubi.tags.DeltaTest,org.apache.kyuubi.tags.HudiTest,org.apache.kyuubi.tags.IcebergTest'
63+
comment: 'verify-spark-3.1'
6164
- java: 8
6265
spark: '3.2'
6366
spark-archive: '-Dspark.archive.mirror=https://dist.apache.org/repos/dist/dev/spark/v3.3.0-rc3-bin -Dspark.archive.name=spark-3.3.0-bin-hadoop3.tgz'
6467
exclude-tags: '-Dmaven.plugin.scalatest.exclude.tags=org.apache.kyuubi.tags.ExtendedSQLTest,org.apache.kyuubi.tags.DeltaTest,org.apache.kyuubi.tags.HudiTest,org.apache.kyuubi.tags.IcebergTest'
68+
comment: 'verify-spark-3.3'
6569
env:
6670
SPARK_LOCAL_IP: localhost
6771
steps:
@@ -98,7 +102,7 @@ jobs:
98102
if: failure()
99103
uses: actions/upload-artifact@v2
100104
with:
101-
name: unit-tests-log
105+
name: unit-tests-log-${{ matrix.java }}-${{ matrix.spark }}-${{ matrix.comment }}
102106
path: |
103107
**/target/unit-tests.log
104108
**/kyuubi-flink-sql-engine.log*
@@ -171,7 +175,7 @@ jobs:
171175
if: failure()
172176
uses: actions/upload-artifact@v2
173177
with:
174-
name: unit-tests-log
178+
name: unit-tests-log-kyuubi-on-k8s-it
175179
path: |
176180
**/target/unit-tests.log
177181
@@ -215,7 +219,7 @@ jobs:
215219
if: failure()
216220
uses: actions/upload-artifact@v2
217221
with:
218-
name: unit-tests-log
222+
name: unit-tests-log-spark-on-k8s-it
219223
path: |
220224
**/target/unit-tests.log
221225
**/kyuubi-spark-sql-engine.log*

0 commit comments

Comments
 (0)