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
docs: clarify docker attach #22611
docs: clarify docker attach #22611
Conversation
LGTM 👍 |
@@ -39,6 +39,14 @@ using `CTRL-p CTRL-q` key sequence. | |||
It is forbidden to redirect the standard input of a `docker attach` command | |||
while attaching to a tty-enabled container (i.e.: launched with `-t`). | |||
|
|||
While attach stream is connected to container's stdio, docker uses ~1MB memory | |||
buffer to maximize the throughput of the application. If this buffer is reached | |||
the speed of the API connection will start to have an effect the application |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/the/on the/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
one nit, but LGTM otherwise |
@@ -39,6 +39,14 @@ using `CTRL-p CTRL-q` key sequence. | |||
It is forbidden to redirect the standard input of a `docker attach` command | |||
while attaching to a tty-enabled container (i.e.: launched with `-t`). | |||
|
|||
While attach stream is connected to container's stdio, docker uses ~1MB memory |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/While attach stream/While an attach stream/?
s/docker uses ~1MB memory buffer/docker uses a ~1MB memory buffer/?
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
LGTM |
👍 But still missing from API docs: https://docs.docker.com/engine/reference/api/docker_remote_api_v1.23/#attach-to-a-container |
Follow-up for #22502
cc @aaronlehmann @cpuguy83
Signed-off-by: Tonis Tiigi tonistiigi@gmail.com