-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dockerfile not up to date #88
Comments
I also tried running via docker directly and first got the contents of First Run, seemingly a lack of First RunFROM node:8-stretch-slim
# Get prerequisites
RUN apt-get update && apt-get install -y \
libcairo2-dev \
man-db
# Put the (symlinked) db in here
RUN mkdir -p /usr/src/sudoroom-data
# Put the app in here
RUN mkdir -p /usr/src/app
WORKDIR /usr/src/app
# Get npm stuff
COPY package.json /usr/src/app/
RUN npm install
# Get everything else
COPY . /usr/src/app
# Expose the port we listen on
EXPOSE 80
# Run it
CMD [ "npm", "start" ]
Second RunFROM node:8-stretch-slim
# Get prerequisites
RUN apt-get update && apt-get install -y \
libcairo2-dev \
man-db \
python \
build-essential
# Put the (symlinked) db in here
RUN mkdir -p /usr/src/sudoroom-data
# Put the app in here
RUN mkdir -p /usr/src/app
WORKDIR /usr/src/app
# Get npm stuff
COPY package.json /usr/src/app/
RUN npm install
# Get everything else
COPY . /usr/src/app
# Expose the port we listen on
EXPOSE 80
# Run it
CMD [ "npm", "start" ]
|
In trying to reuse the existing Dockerfile, this was only possible after
and running with
$ docker run --rm -it -p 8082:80 -v $(pwd)/settings.js:/usr/src/app/settings.js sudoroom/sudo-humans
then.Unfortunately this yields a Node.js error from hell:
which may be due to liberal use of Node 8 and straight updating of all packages in package.json
Is it worth for you to debug this occurrence?
The text was updated successfully, but these errors were encountered: