Skip to content
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

[17.04.x] deprecation and changelog updates #31848

Merged
merged 1 commit into from
Mar 16, 2017

Conversation

thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Mar 15, 2017

  • Interacting with V1 registries is not yet disabled by default, so moving the deprecation version
  • Added various entries to the changelog, and moved some to a different section

ping @vieux PTAL

CHANGELOG.md Outdated
* Wait for OOBE to prevent crashing during host update [#31054](https://github.com/docker/docker/pull/31054)
* Block pulling Windows images on non-Windows daemons [#29001](https://github.com/docker/docker/pull/29001)

### Deprecation

* Removal of the email from `docker login` [#26868](https://github.com/docker/docker/pull/26868)
Copy link
Member Author

Choose a reason for hiding this comment

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

Alternatively, we should revert this change in 17.04 and move it to 17.06

Copy link
Contributor

Choose a reason for hiding this comment

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

right, see #31862

@@ -266,7 +266,7 @@ of the `--changes` flag that allows to pass `Dockerfile` commands.

### Interacting with V1 registries

**Disabled By Default In Release: v17.04**
**Disabled By Default In Release: v17.05**
Copy link
Member Author

Choose a reason for hiding this comment

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

Wondering if we should only deprecate in quarterly releases, i.e. move this to 17.06? If so we should update the deprecation policy accordingly. We can mark features deprecated in monthly releases, but remove them only in quarterly releases wdyt?

Copy link
Member

Choose a reason for hiding this comment

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

Yep.

Copy link
Member Author

Choose a reason for hiding this comment

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

Let me move it to 17.06; we can decide on the change in policy separate from that

@thaJeztah thaJeztah force-pushed the 17.04.0-changelog-updates branch 2 times, most recently from 54ae11a to 9fcee8d Compare March 15, 2017 16:26
@thaJeztah thaJeztah changed the title deprecation updates for 17.04 [17.04.x] deprecation and changelog updates Mar 15, 2017
CHANGELOG.md Outdated
+ Add `--filter enabled=true` for `docker plugin ls` [#28627](https://github.com/docker/docker/pull/28627)
+ Add `--format` to `docker service ls` [#28199](https://github.com/docker/docker/pull/28199)
+ Add `publish` and `expose` filter for `docker ps --filter` [#27557](https://github.com/docker/docker/pull/27557)
* Support multiple service IDs on `docker service ps` [#25234](https://github.com/docker/docker/pull/25234)
+ Allow swarm join with `--availability=drain` [#24993](https://github.com/docker/docker/pull/24993)
* Removal of the email from `docker login` [#26868](https://github.com/docker/docker/pull/26868)
+ Allow `plugin` management commands to address plugins by ID instead of name [#29222](https://github.com/docker/docker/pull/29222)
Copy link
Contributor

Choose a reason for hiding this comment

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

I removed this one already, as @anusha-ragunathan pointed out, the feature was already here, this PR only adds a test

Copy link
Member Author

Choose a reason for hiding this comment

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

oh, right, let me remove the impact/changelog label :)

CHANGELOG.md Outdated
* Wait for OOBE to prevent crashing during host update [#31054](https://github.com/docker/docker/pull/31054)
* Block pulling Windows images on non-Windows daemons [#29001](https://github.com/docker/docker/pull/29001)

### Deprecation
Copy link
Contributor

Choose a reason for hiding this comment

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

@tonistiigi pointed out in the last PR that it's usually used to announce new depreciation, not the actual removal.

Copy link
Member Author

Choose a reason for hiding this comment

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

hm, ok, wondering if it's better to have a separate section to more clearly point it out (it's easily overlooked and may be important when upgrading from a previous version)

open to suggestions though

@thaJeztah thaJeztah force-pushed the 17.04.0-changelog-updates branch 2 times, most recently from 5b7a698 to 5072c7c Compare March 15, 2017 17:40
- Interacting with V1 registries is not yet disabled by default, so moving the deprecation version
- Added various entries to the changelog, and moved some to a different section

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the 17.04.0-changelog-updates branch from 5072c7c to 2387faa Compare March 15, 2017 17:40
@vieux
Copy link
Contributor

vieux commented Mar 15, 2017

LGTM

Copy link
Member

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM 🐯

@vdemeester vdemeester merged commit f15f593 into moby:17.04.x Mar 16, 2017
@thaJeztah thaJeztah deleted the 17.04.0-changelog-updates branch March 16, 2017 09:35
thaJeztah pushed a commit to thaJeztah/docker that referenced this pull request Apr 10, 2017
[17.04.x] deprecation and changelog updates
(cherry picked from commit f15f593)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants