Skip to content

Commit

Permalink
Add Sockets (-H) list to docker -D info.
Browse files Browse the repository at this point in the history
This will allow us to _know_ what the user's -H settings are, which may
be useful for debugging later.

Docker-DCO-1.1-Signed-off-by: Sven Dowideit <[email protected]> (github: SvenDowideit)
  • Loading branch information
SvenDowideit committed Jun 24, 2014
1 parent 664ba0c commit f54823b
Show file tree
Hide file tree
Showing 8 changed files with 38 additions and 15 deletions.
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ feels wrong or incomplete.
When reporting [issues](https://github.com/dotcloud/docker/issues)
on GitHub please include your host OS (Ubuntu 12.04, Fedora 19, etc),
the output of `uname -a` and the output of `docker version` along with
the output of `docker info`. Please include the steps required to reproduce
the output of `docker -D info`. Please include the steps required to reproduce
the problem if possible and applicable.
This information will help us review and fix your issue faster.

Expand Down
3 changes: 3 additions & 0 deletions api/client/commands.go
Original file line number Diff line number Diff line change
Expand Up @@ -452,6 +452,9 @@ func (cli *DockerCli) CmdInfo(args ...string) error {
if initPath := remoteInfo.Get("InitPath"); initPath != "" {
fmt.Fprintf(cli.out, "Init Path: %s\n", initPath)
}
if len(remoteInfo.GetList("Sockets")) != 0 {
fmt.Fprintf(cli.out, "Sockets: %v\n", remoteInfo.GetList("Sockets"))
}
}

if len(remoteInfo.GetList("IndexServerAddress")) != 0 {
Expand Down
2 changes: 2 additions & 0 deletions daemon/daemon.go
Original file line number Diff line number Diff line change
Expand Up @@ -96,6 +96,7 @@ type Daemon struct {
containerGraph *graphdb.Database
driver graphdriver.Driver
execDriver execdriver.Driver
Sockets []string
}

// Install installs daemon capabilities to eng.
Expand Down Expand Up @@ -878,6 +879,7 @@ func NewDaemonFromDirectory(config *daemonconfig.Config, eng *engine.Engine) (*D
sysInitPath: sysInitPath,
execDriver: ed,
eng: eng,
Sockets: config.Sockets,
}

if err := daemon.checkLocaldns(); err != nil {
Expand Down
4 changes: 4 additions & 0 deletions daemonconfig/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ type Config struct {
DisableNetwork bool
EnableSelinuxSupport bool
Context map[string][]string
Sockets []string
}

// ConfigFromJob creates and returns a new DaemonConfig object
Expand Down Expand Up @@ -66,6 +67,9 @@ func ConfigFromJob(job *engine.Job) *Config {
config.Mtu = GetDefaultNetworkMtu()
}
config.DisableNetwork = config.BridgeIface == DisableNetworkBridge
if sockets := job.GetenvList("Sockets"); sockets != nil {
config.Sockets = sockets
}

return config
}
Expand Down
1 change: 1 addition & 0 deletions docker/docker.go
Original file line number Diff line number Diff line change
Expand Up @@ -162,6 +162,7 @@ func main() {
job.Setenv("ExecDriver", *flExecDriver)
job.SetenvInt("Mtu", *flMtu)
job.SetenvBool("EnableSelinuxSupport", *flSelinuxEnabled)
job.SetenvList("Sockets", flHosts.GetAll())
if err := job.Run(); err != nil {
log.Fatal(err)
}
Expand Down
7 changes: 7 additions & 0 deletions docs/sources/reference/api/docker_remote_api_v1.11.md
Original file line number Diff line number Diff line change
Expand Up @@ -1099,9 +1099,16 @@ Display system-wide information
{
"Containers":11,
"Images":16,
"Driver":"btrfs",
"ExecutionDriver":"native-0.1",
"KernelVersion":"3.12.0-1-amd64"
"Debug":false,
"NFd": 11,
"NGoroutines":21,
"NEventsListener":0,
"InitPath":"/usr/bin/docker",
"Sockets":["unix:///var/run/docker.sock"],
"IndexServerAddress":["https://index.docker.io/v1/"],
"MemoryLimit":true,
"SwapLimit":false,
"IPv4Forwarding":true
Expand Down
33 changes: 19 additions & 14 deletions docs/sources/reference/commandline/cli.md
Original file line number Diff line number Diff line change
Expand Up @@ -563,25 +563,30 @@ tar, then the ownerships might not get preserved.

## info

Usage: docker info

Display system-wide information
Usage: docker info

For example:

$ sudo docker info
Containers: 292
Images: 194
$ sudo docker -D info
Containers: 16
Images: 2138
Storage Driver: btrfs
Execution Driver: native-0.1
Kernel Version: 3.12.0-1-amd64
Debug mode (server): false
Debug mode (client): false
Fds: 22
Goroutines: 67
LXC Version: 0.9.0
EventsListeners: 115
Kernel Version: 3.8.0-33-generic
WARNING: No swap limit support

When sending issue reports, please use `docker version` and `docker info` to
Debug mode (client): true
Fds: 16
Goroutines: 104
EventsListeners: 0
Init Path: /usr/bin/docker
Sockets: [unix:///var/run/docker.sock tcp://0.0.0.0:4243]
Username: svendowideit
Registry: [https://index.docker.io/v1/]

The global `-D` option tells all `docker` comands to output debug information.

When sending issue reports, please use `docker version` and `docker -D info` to
ensure we know how your setup is configured.

## inspect
Expand Down
1 change: 1 addition & 0 deletions server/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -780,6 +780,7 @@ func (srv *Server) DockerInfo(job *engine.Job) engine.Status {
v.Set("IndexServerAddress", registry.IndexServerAddress())
v.Set("InitSha1", dockerversion.INITSHA1)
v.Set("InitPath", initPath)
v.SetList("Sockets", srv.daemon.Sockets)
if _, err := v.WriteTo(job.Stdout); err != nil {
return job.Error(err)
}
Expand Down

0 comments on commit f54823b

Please sign in to comment.