diff --git a/components/engine/MAINTAINERS b/components/engine/MAINTAINERS index d81caba80a..a8966873b4 100644 --- a/components/engine/MAINTAINERS +++ b/components/engine/MAINTAINERS @@ -368,7 +368,7 @@ [people.mlaventure] Name = "Kenfe-Mickaƫl Laventure" - Email = "mickael.laventure@docker.com" + Email = "mickael.laventure@gmail.com" GitHub = "mlaventure" [people.moxiegirl] @@ -465,4 +465,3 @@ Name = "Yong Tang" Email = "yong.tang.github@outlook.com" GitHub = "yongtang" - diff --git a/components/engine/api/server/router/container/container_routes.go b/components/engine/api/server/router/container/container_routes.go index d845fdd00f..b7848a3c94 100644 --- a/components/engine/api/server/router/container/container_routes.go +++ b/components/engine/api/server/router/container/container_routes.go @@ -593,7 +593,11 @@ func (s *containerRouter) wsContainersAttach(ctx context.Context, w http.Respons close(done) select { case <-started: - logrus.Errorf("Error attaching websocket: %s", err) + if err != nil { + logrus.Errorf("Error attaching websocket: %s", err) + } else { + logrus.Debug("websocket connection was closed by client") + } return nil default: } diff --git a/components/engine/docs/api/version-history.md b/components/engine/docs/api/version-history.md index 5056c0ddba..2ed2ae5d57 100644 --- a/components/engine/docs/api/version-history.md +++ b/components/engine/docs/api/version-history.md @@ -23,6 +23,18 @@ keywords: "API, Docker, rcli, REST, documentation" configuration is only used for Windows containers. * `GET /containers/(name)/logs` now supports an additional query parameter: `until`, which returns log lines that occurred before the specified timestamp. +* `POST /containers/{id}/exec` now accepts a `WorkingDir` property to set the + work-dir for the exec process, independent of the container's work-dir. +* `Get /version` now returns a `Platform.Name` field, which can be used by products + using Moby as a foundation to return information about the platform. +* `Get /version` now returns a `Components` field, which can be used to return + information about the components used. Information about the engine itself is + now included as a "Component" version, and contains all information from the + top-level `Version`, `GitCommit`, `APIVersion`, `MinAPIVersion`, `GoVersion`, + `Os`, `Arch`, `BuildTime`, `KernelVersion`, and `Experimental` fields. Going + forward, the information from the `Components` section is preferred over their + top-level counterparts. + ## v1.34 API changes diff --git a/components/engine/docs/contributing/set-up-dev-env.md b/components/engine/docs/contributing/set-up-dev-env.md index 28bea5b86f..c2c8389a28 100644 --- a/components/engine/docs/contributing/set-up-dev-env.md +++ b/components/engine/docs/contributing/set-up-dev-env.md @@ -129,13 +129,14 @@ can take over 15 minutes to complete. ```none Successfully built 3d872560918e + Successfully tagged docker-dev:dry-run-test docker run --rm -i --privileged -e BUILDFLAGS -e KEEPBUNDLE -e DOCKER_BUILD_GOGC -e DOCKER_BUILD_PKGS -e DOCKER_CLIENTONLY -e DOCKER_DEBUG -e DOCKER_EXPERIMENTAL -e DOCKER_GITCOMMIT -e DOCKER_GRAPHDRIVER=devicemapper -e DOCKER_INCREMENTAL_BINARY -e DOCKER_REMAP_ROOT -e DOCKER_STORAGE_OPTS -e DOCKER_USERLANDPROXY -e TESTDIRS -e TESTFLAGS -e TIMEOUT -v "home/ubuntu/repos/docker/bundles:/go/src/github.com/moby/moby/bundles" -t "docker-dev:dry-run-test" bash root@f31fa223770f:/go/src/github.com/docker/docker# ``` At this point, your prompt reflects the container's BASH shell. -5. List the contents of the current directory (`/go/src/github.com/moby/moby`). +5. List the contents of the current directory (`/go/src/github.com/docker/docker`). You should see the image's source from the `/go/src/github.com/docker/docker` directory. diff --git a/components/engine/volume/store/store.go b/components/engine/volume/store/store.go index fd1ca616ca..9a511a5df5 100644 --- a/components/engine/volume/store/store.go +++ b/components/engine/volume/store/store.go @@ -145,8 +145,9 @@ func (s *VolumeStore) Purge(name string) { s.globalLock.Lock() v, exists := s.names[name] if exists { - if _, err := volumedrivers.ReleaseDriver(v.DriverName()); err != nil { - logrus.Errorf("Error dereferencing volume driver: %v", err) + driverName := v.DriverName() + if _, err := volumedrivers.ReleaseDriver(driverName); err != nil { + logrus.WithError(err).WithField("driver", driverName).Error("Error releasing reference to volume driver") } } if err := s.removeMeta(name); err != nil {