Skip to content

Commit

Permalink
ci: correct batch etl images
Browse files Browse the repository at this point in the history
  • Loading branch information
cdxker authored and skeptrunedev committed Jan 10, 2025
1 parent d9ad725 commit 42ac341
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
4 changes: 2 additions & 2 deletions batch-etl/Dockerfile.check-completion-worker
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ FROM rust:1.81-slim-bookworm AS chef
# it will be cached from the second build onwards
RUN apt-get update -y && apt-get -y install pkg-config libssl-dev libpq-dev g++ curl
RUN cargo install cargo-chef
WORKDIR app
WORKDIR /app

FROM chef AS planner
COPY . .
Expand All @@ -20,7 +20,7 @@ RUN cargo build --release --features "runtime-env" --bin "check-completion-worke
FROM debian:bookworm-slim as runtime

Check warning on line 20 in batch-etl/Dockerfile.check-completion-worker

View workflow job for this annotation

GitHub Actions / Push Check Completion worker (blacksmith-8vcpu-ubuntu-2204, linux/amd64, latest)

The 'as' keyword should match the case of the 'from' keyword

FromAsCasing: 'as' and 'FROM' keywords' casing do not match More info: https://docs.docker.com/go/dockerfile/rule/from-as-casing/
RUN apt-get update -y && apt-get -y install pkg-config libssl-dev libpq-dev ca-certificates
WORKDIR /app
COPY ./migrations/ /app/migrations
COPY ./ch_migrations/ /app/ch_migrations
COPY --from=builder /app/target/release/check-completion-worker /app/check-completion-worker


Expand Down
4 changes: 2 additions & 2 deletions batch-etl/Dockerfile.create-job-worker
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ FROM rust:1.81-slim-bookworm AS chef
# it will be cached from the second build onwards
RUN apt-get update -y && apt-get -y install pkg-config libssl-dev libpq-dev g++ curl
RUN cargo install cargo-chef
WORKDIR app
WORKDIR /app

FROM chef AS planner
COPY . .
Expand All @@ -20,7 +20,7 @@ RUN cargo build --release --features "runtime-env" --bin "create-job-worker"
FROM debian:bookworm-slim as runtime

Check warning on line 20 in batch-etl/Dockerfile.create-job-worker

View workflow job for this annotation

GitHub Actions / Push Create Job image (blacksmith-8vcpu-ubuntu-2204, linux/amd64, latest)

The 'as' keyword should match the case of the 'from' keyword

FromAsCasing: 'as' and 'FROM' keywords' casing do not match More info: https://docs.docker.com/go/dockerfile/rule/from-as-casing/
RUN apt-get update -y && apt-get -y install pkg-config libssl-dev libpq-dev ca-certificates
WORKDIR /app
COPY ./migrations/ /app/migrations
COPY ./ch_migrations/ /app/ch_migrations
COPY --from=builder /app/target/release/create-job-worker /app/create-job-worker


Expand Down
1 change: 0 additions & 1 deletion batch-etl/Dockerfile.server
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ RUN apt-get update -y; \
;

COPY ./ch_migrations /app/ch_migrations
COPY --from=builder /app/static /app/static
COPY --from=builder /app/target/release/batch-etl-server /app/batch-etl-server

EXPOSE 8090
Expand Down

0 comments on commit 42ac341

Please sign in to comment.