Skip to content

Commit f0644d0

Browse files
authored
Merge pull request #454 from floriantraber/patch-1
documentation fix: init flag is included in docker only after 1.13 not 1.3
2 parents 8cbad29 + 1efc018 commit f0644d0

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

docs/BestPractices.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ Run with `NODE_ENV` set to `production`. This is the way you would pass in secre
1919
```
2020
## Handling Kernel Signals
2121

22-
Node.js was not designed to run as PID 1 which leads to unexpected behaviour when running inside of Docker. For example, a Node.js process running as PID 1 will not respond to `SIGTERM` (`CTRL-C`) and similar signals. As of Docker 1.3, you can use the `--init` flag to wrap your Node.js process with a [lightweight init system](https://github.com/krallin/tini) that properly handles running as PID 1.
22+
Node.js was not designed to run as PID 1 which leads to unexpected behaviour when running inside of Docker. For example, a Node.js process running as PID 1 will not respond to `SIGTERM` (`CTRL-C`) and similar signals. As of Docker 1.13, you can use the `--init` flag to wrap your Node.js process with a [lightweight init system](https://github.com/krallin/tini) that properly handles running as PID 1.
2323

2424
```
2525
docker run -it --init node

0 commit comments

Comments
 (0)