Skip to content

Commit

Permalink
Fix Docker issues
Browse files Browse the repository at this point in the history
  • Loading branch information
darunrs committed Jan 4, 2024
1 parent 968492e commit c21c628
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 3 deletions.
4 changes: 2 additions & 2 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@ services:
- redis
environment:
REDIS_CONNECTION_STRING: redis://redis
LAKE_AWS_ACCESS_KEY:
LAKE_AWS_SECRET_ACCESS_KEY:
AWS_ACCESS_KEY_ID: MOCK
AWS_SECRET_ACCESS_KEY: MOCK
QUEUE_URL: MOCK
START_FROM_BLOCK_QUEUE_URL: MOCK
PORT: 9180
Expand Down
1 change: 1 addition & 0 deletions indexer/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ FROM rust:1.68 AS build
ARG CARGO_BUILD_MODE=release
WORKDIR /tmp/
COPY ./ ./
RUN apt update && apt install -yy protobuf-compiler
RUN if [ "$CARGO_BUILD_MODE" = "debug" ]; then \
cargo build --package queryapi_coordinator; \
else \
Expand Down
5 changes: 4 additions & 1 deletion indexer/queryapi_coordinator/build.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
// TODO: Move to New Coordinator Folder
fn main() -> Result<(), Box<dyn std::error::Error>> {
tonic_build::compile_protos("../../runner/protos/runner.proto")?;
// TODO: Update Runner Env deployment variables and docker compose file to contain RUNNER_PROTO_PATH
let proto_path = std::env::var("RUNNER_PROTO_PATH").unwrap_or_else(|_| "../../runner/protos/runner.proto".to_string());
// TODO: Remove .ok() when we expect tonic build to succeed in deployment
tonic_build::compile_protos(proto_path).ok();
Ok(())
}

0 comments on commit c21c628

Please sign in to comment.