-
Notifications
You must be signed in to change notification settings - Fork 18.7k
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
[Carry 17414] Added additional container information to "docker info". #19249
Conversation
Instead of just showing the number of containers this patch will show the number of running, paused and stopped containers as well. Signed-off-by: Kim Eik <kim@heldig.org> (cherry picked from commit a9804ab)
Vendoring is happening in #19243. |
@@ -112,6 +112,7 @@ This section lists each version from latest to oldest. Each listing includes a | |||
* `GET /networks` now supports filtering by `name`, `id` and `type`. | |||
* `POST /containers/create` now allows you to set the static IPv4 and/or IPv6 address for the container. | |||
* `POST /networks/(id)/connect` now allows you to set the static IPv4 and/or IPv6 address for the container. | |||
* `GET /info` now includes the number of containers running, stopped and paused. |
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.
should have a comma before and paused.
(just a nit)
two really tiny nits, LGTM otherwise |
Signed-off-by: David Calavera <david.calavera@gmail.com>
fixed. |
LGTM moving to "merge", taking previous LGTM's from #17414 into account |
😭 janky, nooooo
|
[Carry 17414] Added additional container information to "docker info".
Carry #17414.
WAIT BEFORE MERGE: This PR is blocked by docker/engine-api#36