Skip to content

Commit

Permalink
apacheGH-43969: [CI][Dev] Prune .dockerignore (apache#43971)
Browse files Browse the repository at this point in the history
### Rationale for this change

The smaller the better, according to https://docs.docker.com/build/cache/optimize/

### What changes are included in this PR?

Prune obsolete or unnecessary inclusions from `.dockerignore`.

### Are these changes tested?

Yes, by CI.

### Are there any user-facing changes?

No.

* GitHub Issue: apache#43969

Authored-by: Antoine Pitrou <antoine@python.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
  • Loading branch information
pitrou authored and zanmato1984 committed Sep 6, 2024
1 parent 967ab83 commit 0d77167
Show file tree
Hide file tree
Showing 13 changed files with 25 additions and 19 deletions.
21 changes: 2 additions & 19 deletions .dockerignore
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,11 @@
# include explicitly
!ci/**
!c_glib/Gemfile
!dev/archery/setup.py
!dev/release/setup-*.sh
!docs/requirements*.txt
!go/go.mod
!go/go.sum
!python/requirements*.txt
!python/manylinux1/**
!r/DESCRIPTION
!ruby/Gemfile
!ruby/red-arrow/Gemfile
Expand All @@ -46,20 +46,3 @@
!ruby/red-parquet/Gemfile
!ruby/red-parquet/lib/parquet/version.rb
!ruby/red-parquet/red-parquet.gemspec
!ruby/red-plasma/Gemfile
!ruby/red-plasma/lib/plasma/version.rb
!ruby/red-plasma/red-plasma.gemspec
!rust/Cargo.toml
!rust/benchmarks/Cargo.toml
!rust/arrow/Cargo.toml
!rust/arrow/benches
!rust/arrow-flight/Cargo.toml
!rust/parquet/Cargo.toml
!rust/parquet/build.rs
!rust/parquet_derive/Cargo.toml
!rust/parquet_derive_test/Cargo.toml
!rust/datafusion/Cargo.toml
!rust/datafusion/benches
!rust/integration-testing/Cargo.toml
!go/go.mod
!go/go.sum
2 changes: 2 additions & 0 deletions .github/workflows/archery.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,14 @@ name: Archery & Crossbow
on:
push:
paths:
- '.dockerignore'
- '.github/workflows/archery.yml'
- 'dev/archery/**'
- 'dev/tasks/**'
- 'docker-compose.yml'
pull_request:
paths:
- '.dockerignore'
- '.github/workflows/archery.yml'
- 'dev/archery/**'
- 'dev/tasks/**'
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/cpp.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ name: C++
on:
push:
paths:
- '.dockerignore'
- '.github/workflows/cpp.yml'
- 'ci/conda_env_*'
- 'ci/docker/**'
Expand All @@ -35,6 +36,7 @@ on:
- 'testing'
pull_request:
paths:
- '.dockerignore'
- '.github/workflows/cpp.yml'
- 'ci/conda_env_*'
- 'ci/docker/**'
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/docs_light.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ name: Docs
on:
pull_request:
paths:
- '.dockerignore'
- 'docs/**'
- '.github/workflows/docs_light.yml'
- 'ci/docker/conda.dockerfile'
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/go.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,13 +20,15 @@ name: Go
on:
push:
paths:
- '.dockerignore'
- '.github/workflows/go.yml'
- 'ci/docker/*_go.dockerfile'
- 'ci/scripts/go_*'
- 'docker-compose.yml'
- 'go/**'
pull_request:
paths:
- '.dockerignore'
- '.github/workflows/go.yml'
- 'ci/docker/*_go.dockerfile'
- 'ci/docker/**'
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ name: Integration
on:
push:
paths:
- '.dockerignore'
- '.github/workflows/integration.yml'
- 'ci/**'
- 'dev/archery/**'
Expand All @@ -33,6 +34,7 @@ on:
- 'format/**'
pull_request:
paths:
- '.dockerignore'
- '.github/workflows/integration.yml'
- 'ci/**'
- 'dev/archery/**'
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/java.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ name: Java
on:
push:
paths:
- '.dockerignore'
- '.github/workflows/java.yml'
- 'ci/docker/*java*'
- 'ci/scripts/java*.sh'
Expand All @@ -29,6 +30,7 @@ on:
- 'java/**'
pull_request:
paths:
- '.dockerignore'
- '.github/workflows/java.yml'
- 'ci/docker/*java*'
- 'ci/scripts/java*.sh'
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/java_jni.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ name: Java JNI
on:
push:
paths:
- '.dockerignore'
- '.github/workflows/java_jni.yml'
- 'ci/docker/**'
- 'ci/scripts/cpp_build.sh'
Expand All @@ -29,6 +30,7 @@ on:
- 'java/**'
pull_request:
paths:
- '.dockerignore'
- '.github/workflows/java_jni.yml'
- 'ci/docker/**'
- 'ci/scripts/cpp_build.sh'
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/js.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,14 @@ name: NodeJS
on:
push:
paths:
- '.dockerignore'
- '.github/workflows/js.yml'
- 'ci/docker/*js.dockerfile'
- 'ci/scripts/js_*'
- 'js/**'
pull_request:
paths:
- '.dockerignore'
- '.github/workflows/js.yml'
- 'ci/docker/*js.dockerfile'
- 'ci/scripts/js_*'
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/python.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,13 +20,15 @@ name: Python
on:
push:
paths:
- '.dockerignore'
- '.github/workflows/python.yml'
- 'ci/**'
- 'cpp/**'
- 'docker-compose.yml'
- 'python/**'
pull_request:
paths:
- '.dockerignore'
- '.github/workflows/python.yml'
- 'ci/**'
- 'cpp/**'
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/r.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ name: R
on:
push:
paths:
- '.dockerignore'
- ".github/workflows/r.yml"
- "ci/docker/**"
- "ci/etc/rprofile"
Expand All @@ -32,6 +33,7 @@ on:
- "r/**"
pull_request:
paths:
- '.dockerignore'
- ".github/workflows/r.yml"
- "ci/docker/**"
- "ci/etc/rprofile"
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/ruby.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ name: C GLib & Ruby
on:
push:
paths:
- '.dockerignore'
- '.github/workflows/ruby.yml'
- 'ci/docker/**'
- 'ci/scripts/c_glib_*'
Expand All @@ -33,6 +34,7 @@ on:
- 'ruby/**'
pull_request:
paths:
- '.dockerignore'
- '.github/workflows/ruby.yml'
- 'ci/docker/**'
- 'ci/scripts/c_glib_*'
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/swift.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,13 +20,15 @@ name: Swift
on:
push:
paths:
- '.dockerignore'
- '.github/workflows/swift.yml'
- 'ci/docker/*swift*'
- 'ci/scripts/swift_*'
- 'docker-compose.yml'
- 'swift/**'
pull_request:
paths:
- '.dockerignore'
- '.github/workflows/swift.yml'
- 'ci/docker/*swift*'
- 'ci/scripts/swift_*'
Expand Down

0 comments on commit 0d77167

Please sign in to comment.