Skip to content

Commit

Permalink
Merge pull request docker#54 from klintzz/patch-1
Browse files Browse the repository at this point in the history
Dockerfile CMD path fix
  • Loading branch information
StefanScherer authored Jul 8, 2020
2 parents d4da770 + 631465e commit 12a64fa
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
10 changes: 5 additions & 5 deletions docs/tutorial/image-building-best-practices/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ command, you can see the command that was used to create each layer within an im

```plaintext
IMAGE CREATED CREATED BY SIZE COMMENT
a78a40cbf866 18 seconds ago /bin/sh -c #(nop) CMD ["node" "/app/src/ind… 0B
a78a40cbf866 18 seconds ago /bin/sh -c #(nop) CMD ["node" "src/index.j… 0B
f1d1808565d6 19 seconds ago /bin/sh -c yarn install --production 85.4MB
a2c054d14948 36 seconds ago /bin/sh -c #(nop) COPY dir:5dc710ad87c789593… 198kB
9577ae713121 37 seconds ago /bin/sh -c #(nop) WORKDIR /app 0B
Expand Down Expand Up @@ -56,7 +56,7 @@ FROM node:12-alpine
WORKDIR /app
COPY . .
RUN yarn install --production
CMD ["node", "/app/src/index.js"]
CMD ["node", "src/index.js"]
```
Going back to the image history output, we see that each command in the Dockerfile becomes a new layer in the image.
Expand All @@ -76,7 +76,7 @@ a change to the `package.json`. Make sense?
COPY package.json yarn.lock ./
RUN yarn install --production
COPY . .
CMD ["node", "/app/src/index.js"]
CMD ["node", "src/index.js"]
```

1. Create a file named `.dockerignore` in the same folder as the Dockerfile with the following contents.
Expand Down Expand Up @@ -125,7 +125,7 @@ a change to the `package.json`. Make sense?
---> 4e68fbc2d704
Step 5/6 : COPY . .
---> a239a11f68d8
Step 6/6 : CMD ["node", "/app/src/index.js"]
Step 6/6 : CMD ["node", "src/index.js"]
---> Running in 49999f68df8f
Removing intermediate container 49999f68df8f
---> e709c03bc597
Expand Down Expand Up @@ -154,7 +154,7 @@ a change to the `package.json`. Make sense?
---> 4e68fbc2d704
Step 5/6 : COPY . .
---> cccde25a3d9a
Step 6/6 : CMD ["node", "/app/src/index.js"]
Step 6/6 : CMD ["node", "src/index.js"]
---> Running in 2be75662c150
Removing intermediate container 2be75662c150
---> 458e5c6f080c
Expand Down
2 changes: 1 addition & 1 deletion docs/tutorial/our-application/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ see a few flaws in the Dockerfile below. But, don't worry! We'll go over them.
WORKDIR /app
COPY . .
RUN yarn install --production
CMD ["node", "/app/src/index.js"]
CMD ["node", "src/index.js"]
```

Please check that the file `Dockerfile` has no file extension like `.txt`. Some editors may append this file extension automatically and this would result in an error in the next step.
Expand Down

0 comments on commit 12a64fa

Please sign in to comment.