[17.10] add 17.10 changelog #256

vieux · 2017-09-28T00:30:23Z

No description provided.

vieux · 2017-09-28T00:33:56Z

thaJeztah

Comments inline

thaJeztah · 2017-09-28T12:55:53Z

CHANGELOG

+### Builder
+
+* Reset uid/gid to 0 in build context to fix cache busting issues on `ADD/COPY` [docker/cli#513](https://github.com/docker/cli/pull/513)
+- Fix build with `ADD` urls without any sub path [moby/moby#34217](https://github.com/moby/moby/pull/34217)


I think we can change this to Add support for .., as it wasn't really a bug (imo), more an enhancement

thaJeztah · 2017-09-28T12:57:09Z

CHANGELOG

+
+* Reset uid/gid to 0 in build context to fix cache busting issues on `ADD/COPY` [docker/cli#513](https://github.com/docker/cli/pull/513)
+- Fix build with `ADD` urls without any sub path [moby/moby#34217](https://github.com/moby/moby/pull/34217)
+- Fix loader error with different build syntax [docker/cli#544](https://github.com/docker/cli/pull/544)


This was a fix for docker stack deploy with a compose file that had a build-section, may have to be moved to a different section

/cc @vdemeester

thaJeztah · 2017-09-28T12:57:32Z

CHANGELOG

+### Client
+
+* Move output of `docker stack rm` to stdout [docker/cli#491](https://github.com/docker/cli/pull/491)
+* Remove deprecated `--enable-api-cors` flag [moby/moby#34821](https://github.com/moby/moby/pull/34821)


Should this be in a Deprecations section?

thaJeztah · 2017-09-28T12:58:33Z

CHANGELOG

+
+* Move output of `docker stack rm` to stdout [docker/cli#491](https://github.com/docker/cli/pull/491)
+* Remove deprecated `--enable-api-cors` flag [moby/moby#34821](https://github.com/moby/moby/pull/34821)
+* Use nature sort secrets and configs in cli [docker/cli#307](https://github.com/docker/cli/pull/307)


Saw this in the PR, but s/nature/natural/ is probably better;

* Use natural sort for secrets and configs [docker/cli#307](https://github.com/docker/cli/pull/307)

thaJeztah · 2017-09-28T12:59:26Z

CHANGELOG

+* Move output of `docker stack rm` to stdout [docker/cli#491](https://github.com/docker/cli/pull/491)
+* Remove deprecated `--enable-api-cors` flag [moby/moby#34821](https://github.com/moby/moby/pull/34821)
+* Use nature sort secrets and configs in cli [docker/cli#307](https://github.com/docker/cli/pull/307)
+* Use non-detached mode as default for `docker service` commands [docker/cli#525](https://github.com/docker/cli/pull/525)


Do you think we should put a note/warning/message at the top of the changelog for this to emphasise this change?

thaJeztah · 2017-09-28T13:15:22Z

CHANGELOG

+* LCOW: Prepare work for bind mounts [moby/moby#34258](https://github.com/moby/moby/pull/34258)
+* LCOW: Support for docker cp, ADD/COPY on build [moby/moby#34252](https://github.com/moby/moby/pull/34252)
+* LCOW: VHDX boot to readonly. [moby/moby#34754](https://github.com/moby/moby/pull/34754)
+- Fix volume inspect with empty ID [moby/moby#34770](https://github.com/moby/moby/pull/34770)


Looks like this was cherry-picked into 17.09 7034626#diff-65244c7ec61daa507c915173882e5de3R215 (only in components/cli, but it looks like a client fix)

/cc @dnephin

Yes, I agree this fix is already in 17.09.

thaJeztah · 2017-09-28T13:17:59Z

CHANGELOG

+* LCOW: VHDX boot to readonly. [moby/moby#34754](https://github.com/moby/moby/pull/34754)
+- Fix volume inspect with empty ID [moby/moby#34770](https://github.com/moby/moby/pull/34770)
+- Fix variable shadowing causing LayersSize to be reported as 0 [moby/moby#34826](https://github.com/moby/moby/pull/34826)
+* Volume: evaluate symlinks before relabeling mount source [moby/moby#34792](https://github.com/moby/moby/pull/34792)


Should we mention SELinux here?

thaJeztah · 2017-09-28T13:19:00Z

CHANGELOG

+- Fix volume inspect with empty ID [moby/moby#34770](https://github.com/moby/moby/pull/34770)
+- Fix variable shadowing causing LayersSize to be reported as 0 [moby/moby#34826](https://github.com/moby/moby/pull/34826)
+* Volume: evaluate symlinks before relabeling mount source [moby/moby#34792](https://github.com/moby/moby/pull/34792)
+* Set selinux label on local volumes from mounts API [moby/moby#34684](https://github.com/moby/moby/pull/34684)


This was back ported to 17.09 through #241, so not new

thaJeztah · 2017-09-28T13:19:39Z

CHANGELOG

+- Fix variable shadowing causing LayersSize to be reported as 0 [moby/moby#34826](https://github.com/moby/moby/pull/34826)
+* Volume: evaluate symlinks before relabeling mount source [moby/moby#34792](https://github.com/moby/moby/pull/34792)
+* Set selinux label on local volumes from mounts API [moby/moby#34684](https://github.com/moby/moby/pull/34684)
+* Close pipe in overlay2 graphdriver [moby/moby#34863](https://github.com/moby/moby/pull/34863)


This was back ported to 17.09 through #245, so not new

thaJeztah · 2017-09-28T13:22:49Z

CHANGELOG

+
+### Swarm Mode
+
+* Use limit with `docker swarm init --force-new-cluster` [moby/moby#34881](https://github.com/moby/moby/pull/34881)


Perhaps

* Produce an error if `docker swarm init --force-new-cluster` is executed on worker nodes [moby/moby#34881](https://github.com/moby/moby/pull/34881)

tonistiigi · 2017-09-28T00:47:59Z

CHANGELOG

+
+### Builder
+
+* Reset uid/gid to 0 in build context to fix cache busting issues on `ADD/COPY` [docker/cli#513](https://github.com/docker/cli/pull/513)


Reset uid/gid to 0 in uploaded build context to share build cache with other clients

thaJeztah · 2017-09-28T19:18:28Z

docker/cli#565 was just merged, and needs an entry if we didn't cut off the 17.10 branch yet;

* Change the default output format for `docker container stats` to show `CONTAINER ID` and `NAME` [docker/cli#565](https://github.com/docker/cli/pull/565)
+ Add `--no-trunc` flag to `docker container stats` [docker/cli#565](https://github.com/docker/cli/pull/565)

dnephin · 2017-09-28T19:26:59Z

docker trust is in experimental for 17.10 , right? Should we mention it?

thaJeztah · 2017-09-28T19:27:37Z

Ah, yes, that should definitely be added, thanks!

vieux · 2017-09-29T06:03:02Z

@thaJeztah I addressed most of your comments, feel free to edit the rest directly if you want, I wasn't sure what to do, otherwise, I think it's good for RC1

Signed-off-by: Victor Vieux <victorvieux@gmail.com>

Signed-off-by: Andrew Hsu <andrewhsu@docker.com>

vieux · 2017-09-29T22:56:40Z

LGTM

tonistiigi · 2017-09-29T22:58:34Z

LGTM

andrewhsu · 2017-10-04T21:07:58Z

cc @JimGalasyn

andrewhsu

LGTM

mdlinville · 2017-10-04T21:58:33Z

@andrewhsu FYI CE stuff is me and @gbarr01 :)

Merged with https://github.com/seemethere/unir Upstream-commit: 8233fbfa68896527ffaef157eb0728c71712d27e Component: packaging

…it_grpc [19.03 backport] Increase max recv gRPC message size for nodes and secrets Upstream-commit: 746dce1994c3062d9c477f29e0578acf4f1da310 Component: engine

andrewhsu requested a review from joaofnfernandes September 28, 2017 00:37

thaJeztah suggested changes Sep 28, 2017

View reviewed changes

tonistiigi reviewed Sep 28, 2017

View reviewed changes

andrewhsu changed the base branch from master to 17.10 September 29, 2017 22:42

vieux and others added 3 commits September 29, 2017 15:47

add 17.10 changelog

71bb1e8

Signed-off-by: Victor Vieux <victorvieux@gmail.com>

update punctuation and spacing

db0a220

Signed-off-by: Andrew Hsu <andrewhsu@docker.com>

removed deprecation line

079f5eb

Signed-off-by: Andrew Hsu <andrewhsu@docker.com>

andrewhsu added this to the 17.10.0 milestone Sep 29, 2017

vieux changed the title ~~[WIP] [17.10] add 17.10 changelog~~ [17.10] add 17.10 changelog Sep 30, 2017

andrewhsu approved these changes Oct 4, 2017

View reviewed changes

andrewhsu merged commit 8ab000a into docker-archive:17.10 Oct 4, 2017

vieux deleted the 17.10-changelog branch October 4, 2017 22:30

docker-jenkins pushed a commit that referenced this pull request Nov 15, 2018

Merge pull request #256 from thaJeztah/18.09_backport_fedora29

497150a

Merged with https://github.com/seemethere/unir Upstream-commit: 8233fbfa68896527ffaef157eb0728c71712d27e Component: packaging

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[17.10] add 17.10 changelog #256

[17.10] add 17.10 changelog #256

vieux commented Sep 28, 2017 •

edited

Loading

vieux commented Sep 28, 2017

thaJeztah left a comment

thaJeztah Sep 28, 2017

thaJeztah Sep 28, 2017

thaJeztah Sep 28, 2017

thaJeztah Sep 28, 2017

thaJeztah Sep 28, 2017

thaJeztah Sep 28, 2017

dnephin Sep 28, 2017

thaJeztah Sep 28, 2017

thaJeztah Sep 28, 2017

thaJeztah Sep 28, 2017

thaJeztah Sep 28, 2017

tonistiigi Sep 28, 2017

thaJeztah commented Sep 28, 2017

dnephin commented Sep 28, 2017

thaJeztah commented Sep 28, 2017

vieux commented Sep 29, 2017

vieux commented Sep 29, 2017

tonistiigi commented Sep 29, 2017

andrewhsu commented Oct 4, 2017

andrewhsu left a comment

mdlinville commented Oct 4, 2017


		### Swarm Mode

		* Use limit with `docker swarm init --force-new-cluster` [moby/moby#34881](https://github.com/moby/moby/pull/34881)


		### Builder

		* Reset uid/gid to 0 in build context to fix cache busting issues on `ADD/COPY` [docker/cli#513](https://github.com/docker/cli/pull/513)

[17.10] add 17.10 changelog #256

[17.10] add 17.10 changelog #256

Conversation

vieux commented Sep 28, 2017 • edited Loading

vieux commented Sep 28, 2017

thaJeztah left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

thaJeztah commented Sep 28, 2017

dnephin commented Sep 28, 2017

thaJeztah commented Sep 28, 2017

vieux commented Sep 29, 2017

vieux commented Sep 29, 2017

tonistiigi commented Sep 29, 2017

andrewhsu commented Oct 4, 2017

andrewhsu left a comment

Choose a reason for hiding this comment

mdlinville commented Oct 4, 2017

vieux commented Sep 28, 2017 •

edited

Loading