Skip to content
This repository has been archived by the owner on Feb 11, 2024. It is now read-only.

Commit

Permalink
fix: add protoc to docker build (#11)
Browse files Browse the repository at this point in the history
  • Loading branch information
Xavier Basty authored Mar 29, 2023
1 parent 0f4fef4 commit 23636fc
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
3 changes: 0 additions & 3 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -122,9 +122,6 @@ jobs:
- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v2

- name: Install Protoc
uses: arduino/setup-protoc@v1

- name: Build, tag, and push image
uses: docker/build-push-action@v3
with:
Expand Down
6 changes: 6 additions & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,12 @@ FROM chef AS build
ARG release
ENV RELEASE=${release:+--release}

# This is a build requirement of `opentelemetry-otlp`. Once the new version
# is rolled out, which no longer requires the `protoc`, we'll be able to
# get rid of this.
RUN apt-get update \
&& apt-get install -y --no-install-recommends protobuf-compiler

WORKDIR /app
# Cache dependancies
COPY --from=plan /app/recipe.json recipe.json
Expand Down

0 comments on commit 23636fc

Please sign in to comment.