Skip to content

Commit 6bbec6f

Browse files
committed
ci: use path allow list for iceberg workflow triggers
Switch the iceberg_spark_test.yml push and pull_request triggers from paths-ignore (deny list) to paths (allow list), so the workflow runs only when files that actually affect iceberg tests change. With a deny list, every new file type that should not trigger iceberg CI must be added explicitly. An allow list excludes everything by default, so unrelated changes such as dev/ scripts, release tooling, and golden-file regeneration no longer trigger iceberg CI. The allow list mirrors spark_sql_test.yml, scoped to dev/diffs/iceberg and the iceberg-specific workflow and setup action.
1 parent 2a266f6 commit 6bbec6f

1 file changed

Lines changed: 34 additions & 20 deletions

File tree

.github/workflows/iceberg_spark_test.yml

Lines changed: 34 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -25,27 +25,41 @@ on:
2525
push:
2626
branches:
2727
- main
28-
paths-ignore:
29-
- "benchmarks/**"
30-
- "doc/**"
31-
- "docs/**"
32-
- "**.md"
33-
- "native/core/benches/**"
34-
- "native/spark-expr/benches/**"
35-
- "spark/src/test/**"
36-
- "spark/src/main/scala/org/apache/comet/GenerateDocs.scala"
37-
- "spark-integration/**"
28+
paths:
29+
- "native/**/src/**"
30+
- "native/**/Cargo.toml"
31+
- "native/Cargo.lock"
32+
- "!native/hdfs/**"
33+
- "!native/fs-hdfs/**"
34+
- "common/src/main/**"
35+
- "common/pom.xml"
36+
- "spark/src/main/**"
37+
- "!spark/src/main/scala/org/apache/comet/GenerateDocs.scala"
38+
- "spark/pom.xml"
39+
- "dev/diffs/iceberg/**"
40+
- "pom.xml"
41+
- "rust-toolchain.toml"
42+
- ".github/workflows/iceberg_spark_test.yml"
43+
- ".github/actions/setup-builder/**"
44+
- ".github/actions/setup-iceberg-builder/**"
3845
pull_request:
39-
paths-ignore:
40-
- "benchmarks/**"
41-
- "doc/**"
42-
- "docs/**"
43-
- "**.md"
44-
- "native/core/benches/**"
45-
- "native/spark-expr/benches/**"
46-
- "spark/src/test/**"
47-
- "spark/src/main/scala/org/apache/comet/GenerateDocs.scala"
48-
- "spark-integration/**"
46+
paths:
47+
- "native/**/src/**"
48+
- "native/**/Cargo.toml"
49+
- "native/Cargo.lock"
50+
- "!native/hdfs/**"
51+
- "!native/fs-hdfs/**"
52+
- "common/src/main/**"
53+
- "common/pom.xml"
54+
- "spark/src/main/**"
55+
- "!spark/src/main/scala/org/apache/comet/GenerateDocs.scala"
56+
- "spark/pom.xml"
57+
- "dev/diffs/iceberg/**"
58+
- "pom.xml"
59+
- "rust-toolchain.toml"
60+
- ".github/workflows/iceberg_spark_test.yml"
61+
- ".github/actions/setup-builder/**"
62+
- ".github/actions/setup-iceberg-builder/**"
4963
# manual trigger
5064
# https://docs.github.com/en/actions/managing-workflow-runs/manually-running-a-workflow
5165
workflow_dispatch:

0 commit comments

Comments
 (0)