Skip to content
This repository was archived by the owner on Oct 13, 2023. It is now read-only.

[18.01] Changelog for 18.01#379

Merged
andrewhsu merged 1 commit into
docker-archive:18.01from
jose-bigio:18.01_changelog
Jan 4, 2018
Merged

[18.01] Changelog for 18.01#379
andrewhsu merged 1 commit into
docker-archive:18.01from
jose-bigio:18.01_changelog

Conversation

@jose-bigio
Copy link
Copy Markdown

Ping @andrewhsu @vieux @thaJeztah

Signed-off-by: jose-bigio jose.bigio@docker.com

@jose-bigio jose-bigio changed the title Changelog for 18.01 [18.01] Changelog for 18.01 Jan 3, 2018
@jose-bigio jose-bigio added this to the 18.01.0 milestone Jan 3, 2018
@andrewhsu andrewhsu requested review from thaJeztah and vieux January 3, 2018 23:08
Comment thread CHANGELOG.md Outdated
### ---Triage---

* Legacy images replacement with multi-arch ones on AArch64 [moby/moby#35773](https://github.com/moby/moby/pull/35773)
* Vendor: update archive/tar [moby/moby#35728](https://github.com/moby/moby/pull/35728)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd remove all the vendor

Comment thread CHANGELOG.md Outdated

### ---Triage---

* Legacy images replacement with multi-arch ones on AArch64 [moby/moby#35773](https://github.com/moby/moby/pull/35773)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i'd remove this one

Comment thread CHANGELOG.md Outdated
* Return errors from client in stack deploy configs [docker/cli#757](https://github.com/docker/cli/pull/757)
- Fix description of filter flag in prune commands [docker/cli#774](https://github.com/docker/cli/pull/774)
+ Add kubernetes support to docker/cli [docker/cli#721](https://github.com/docker/cli/pull/721)
- Fix #512 Bash autocompletion works incorrect with inspect [docker/cli#713](https://github.com/docker/cli/pull/713)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i'd remove "Fix #512"

@vieux
Copy link
Copy Markdown
Contributor

vieux commented Jan 4, 2018

thanks @jose-bigio , as I said I would remove all the vendor update and I would use

``

around flags and command such as docker exec

Comment thread CHANGELOG.md Outdated

### Builder

* Remove dead code and redundant build tags [moby/moby#35823](https://github.com/moby/moby/pull/35823)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not really worth mentioning

Comment thread CHANGELOG.md Outdated
### Builder

* Remove dead code and redundant build tags [moby/moby#35823](https://github.com/moby/moby/pull/35823)
* Skip container ID remapping, if the file is overlayfs whiteout. [moby/moby#35822](https://github.com/moby/moby/pull/35822)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perhaps;

Fix files not being deleted during if user-namespaces are enabled

Comment thread CHANGELOG.md Outdated

* Remove dead code and redundant build tags [moby/moby#35823](https://github.com/moby/moby/pull/35823)
* Skip container ID remapping, if the file is overlayfs whiteout. [moby/moby#35822](https://github.com/moby/moby/pull/35822)
- Fix environ substitutions in `docker commit --change ...` [moby/moby#35582](https://github.com/moby/moby/pull/35582)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perhaps:

Add support for expanding environment-variables in `docker commit --change ...`

Comment thread CHANGELOG.md Outdated

### Client

* Remove reference to non-existing "stack config" command [docker/cli#765](https://github.com/docker/cli/pull/765)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no need to mention this one

Comment thread CHANGELOG.md Outdated
* Remove reference to non-existing "stack config" command [docker/cli#765](https://github.com/docker/cli/pull/765)
* Return errors from client in stack deploy configs [docker/cli#757](https://github.com/docker/cli/pull/757)
- Fix description of filter flag in prune commands [docker/cli#774](https://github.com/docker/cli/pull/774)
+ Add kubernetes support to docker/cli [docker/cli#721](https://github.com/docker/cli/pull/721)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably be under a Experimental header, and we need to make clear it's only for docker stack commands; perhaps;

Add experimental support for docker stack commands on kubernetes

(better suggestions welcome)

Comment thread CHANGELOG.md Outdated
* Carry #35609 (remove import of opencontainers/runc in windows) [moby/moby#35788](https://github.com/moby/moby/pull/35788)
* Devmapper: Log fstype and mount options during mount error [moby/moby#35732](https://github.com/moby/moby/pull/35732)
* Windows: Case-insensitive filename matching against builder cache [moby/moby#35793](https://github.com/moby/moby/pull/35793)
- Fix race conditions in libcontainerd process handling [moby/moby#35809](https://github.com/moby/moby/pull/35809)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was already back ported to 17.12

Comment thread CHANGELOG.md Outdated

* Carry #35609 (remove import of opencontainers/runc in windows) [moby/moby#35788](https://github.com/moby/moby/pull/35788)
* Devmapper: Log fstype and mount options during mount error [moby/moby#35732](https://github.com/moby/moby/pull/35732)
* Windows: Case-insensitive filename matching against builder cache [moby/moby#35793](https://github.com/moby/moby/pull/35793)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was already back ported to 17.12

Comment thread CHANGELOG.md Outdated

### ---Triage---

* Legacy images replacement with multi-arch ones on AArch64 [moby/moby#35773](https://github.com/moby/moby/pull/35773)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no need to mention this one

Comment thread CHANGELOG.md Outdated
### ---Triage---

* Legacy images replacement with multi-arch ones on AArch64 [moby/moby#35773](https://github.com/moby/moby/pull/35773)
* Vendor: update archive/tar [moby/moby#35728](https://github.com/moby/moby/pull/35728)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already back ported to 17.12, and not worth mentioning

Comment thread CHANGELOG.md Outdated
### --Delete-- (For review only)
- Fixed typo DEREPCATED -> DEPRECATED. [moby/moby#35899](https://github.com/moby/moby/pull/35899)
- Fix typo in link [docker/cli#759](https://github.com/docker/cli/pull/759)
* Bump API version to 1.36 [moby/moby#35738](https://github.com/moby/moby/pull/35738)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reverted in #378

note that API version bumps should normally be included in the changelog. We can copy the exact changes from the API history; https://github.com/docker/docker-ce/blob/17.12/components/engine/docs/api/version-history.md#v135-api-changes

There's no API changes in 18.01, so not for this release 😄

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything under the delete header I intended to delete.
Do you agree with that?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, they are all ok to remove (just pointed out the API things for future change logs 😅 )

@thaJeztah
Copy link
Copy Markdown
Member

Oops, race condition; left some comments on now outdated lines

@vieux some of the vendor updates included fixes that are worth mentioning; I pointed those out in my review 😅

@jose-bigio
Copy link
Copy Markdown
Author

Thanks for all the comments @thaJeztah I'll make sure to make the changes!

Copy link
Copy Markdown
Contributor

@andrewhsu andrewhsu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if you could squish commits, LGTM

Signed-off-by: jose-bigio <jose.bigio@docker.com>
@andrewhsu andrewhsu merged commit f0e6477 into docker-archive:18.01 Jan 4, 2018
Comment thread CHANGELOG.md

### Runtime

* Re-validate Mounts on container start [moby/moby#35833](https://github.com/moby/moby/pull/35833)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like you missed this one; #379 (comment)

Comment thread CHANGELOG.md
* Re-validate Mounts on container start [moby/moby#35833](https://github.com/moby/moby/pull/35833)
- Fix overlay2 storage driver inside a user namespace [moby/moby#35794](https://github.com/moby/moby/pull/35794)
* Zfs: fix busy error on container stop [moby/moby#35674](https://github.com/moby/moby/pull/35674)
- Fix #35843 regression on health check workingdir [moby/moby#35845](https://github.com/moby/moby/pull/35845)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like you missed this one #379 (comment)

cpuguy83 pushed a commit to cpuguy83/docker-ce that referenced this pull request Feb 7, 2018
…gin-ls-filter

Add metric plugins to zsh completion for `plugin ls --filter capability`
Upstream-commit: 915b18bdc9f08e279bb7c1044c4c86be80168ad6
Component: cli
docker-jenkins pushed a commit that referenced this pull request Sep 24, 2019
[18.09 backport] Fix unmount redeclaration on darwin
Upstream-commit: 10ca83725886b3804b5ba53f26678877104ec96d
Component: engine
docker-jenkins pushed a commit that referenced this pull request Sep 25, 2019
[19.03 backport] RPM spec: remove -ce suffix from distribution_based_engine JSON
Upstream-commit: 146a09e4a18213e28cad8743e10c1639ac7827cc
Component: packaging
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants