-
Notifications
You must be signed in to change notification settings - Fork 16
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
1 changed file
with
16 additions
and
7 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,14 +1,23 @@ | ||
FROM node:6.10 | ||
|
||
COPY *.* /app/ | ||
COPY views/ /app/ | ||
COPY public /app/ | ||
LABEL maintainer "[email protected]" | ||
LABEL version="1.0" | ||
LABEL description="If you need to convert text from one markup format into another, pandoc is your swiss-army knife." | ||
|
||
WORKDIR /app/ | ||
# Create app directory | ||
RUN mkdir -p /usr/src/pandoc-as-a-service | ||
COPY . /usr/src/pandoc-as-a-service | ||
WORKDIR /usr/src/pandoc-as-a-service | ||
|
||
RUN apt-get update \ | ||
&& apt-get install -y pandoc \ | ||
&& npm install | ||
# Install packages | ||
RUN apt-get update --fix-missing | ||
RUN apt-get install -y pandoc | ||
|
||
# Clean up | ||
RUN apt-get clean && rm -rf /var/lib/apt/lists/* | ||
|
||
# Install service | ||
RUN npm install | ||
This comment has been minimized.
Sorry, something went wrong.
This comment has been minimized.
Sorry, something went wrong. |
||
|
||
EXPOSE 8080 | ||
|
||
|
Splitting everything up into multiple run statements is a bad idea. Docker uses file system diffing for layer-to-layer differences. So if you delete something your container crows as much as it would grow when you add it.
Some tips for small images: https://www.shivering-isles.com/docker-minimize-your-containers-with-alpine-linux/#size