Skip to content

Commit caf289f

Browse files
ci: install the interop job's build dependencies directly
The job runs on a plain runner rather than the amd64/rust container, so setup-builder's apt-get calls fail there. Install the protobuf compiler and add rustfmt, which is what the Substrait build needs from it.
1 parent b0a7448 commit caf289f

1 file changed

Lines changed: 8 additions & 3 deletions

File tree

‎.github/workflows/rust.yml‎

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -720,11 +720,16 @@ jobs:
720720
substrait:
721721
- 'datafusion/substrait/**'
722722
- '.github/workflows/rust.yml'
723+
# `setup-builder` assumes the amd64/rust container this job does not use,
724+
# so the two things the Substrait build needs are installed directly.
725+
- name: Install protobuf compiler
726+
if: steps.filter.outputs.substrait == 'true'
727+
run: |
728+
sudo apt-get update
729+
sudo apt-get install -y protobuf-compiler
723730
- name: Setup Rust toolchain
724731
if: steps.filter.outputs.substrait == 'true'
725-
uses: ./.github/actions/setup-builder
726-
with:
727-
rust-version: stable
732+
run: rustup component add rustfmt
728733
- name: Setup Java
729734
if: steps.filter.outputs.substrait == 'true'
730735
uses: actions/setup-java@de7274f081f381c8f8158605e0321c36c376e2e6 # v6

0 commit comments

Comments
 (0)