Skip to content

Commit

Permalink
Fix nodejs issues in Docker build (#731)
Browse files Browse the repository at this point in the history
Co-authored-by: Jad Naous <[email protected]>
  • Loading branch information
jnaous and Jad Naous authored May 29, 2023
1 parent 10e021a commit b70b372
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
4 changes: 2 additions & 2 deletions ballista/scheduler/ui/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@
"web-vitals": "^1.0.1"
},
"scripts": {
"start": "react-scripts start",
"build": "react-scripts build",
"start": "react-scripts --openssl-legacy-provider start",
"build": "react-scripts --openssl-legacy-provider build",
"test": "react-scripts test",
"eject": "react-scripts eject"
},
Expand Down
7 changes: 5 additions & 2 deletions dev/docker/ballista-builder.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,11 @@ ENV RUST_BACKTRACE=full
ENV DEBIAN_FRONTEND=noninteractive

RUN apt-get update && \
apt-get -y install libssl-dev openssl zlib1g zlib1g-dev libpq-dev cmake protobuf-compiler netcat curl unzip \
nodejs npm && \
apt-get -y install libssl-dev openssl zlib1g zlib1g-dev libpq-dev cmake protobuf-compiler netcat curl unzip

RUN curl -fsSL https://deb.nodesource.com/setup_18.x | bash - && \
apt-get update && \
apt-get install -y nodejs && \
npm install -g yarn

# create build user with same UID as
Expand Down

0 comments on commit b70b372

Please sign in to comment.