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

[17.12] backport fix plural singular node generic resources#404

Merged
andrewhsu merged 2 commits intodocker-archive:17.12from
thaJeztah:17.12-backport-fix-plural-singular-node-generic-resources
Feb 13, 2018
Merged

[17.12] backport fix plural singular node generic resources#404
andrewhsu merged 2 commits intodocker-archive:17.12from
thaJeztah:17.12-backport-fix-plural-singular-node-generic-resources

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

backport of moby/moby#35970 and moby/moby#36125 for 17.12

Renaud Gaubert and others added 2 commits January 30, 2018 14:44
Signed-off-by: Renaud Gaubert <rgaubert@nvidia.com>
(cherry picked from commit 6702ac5)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Daemon flags that can be specified multiple times use
singlar names for flags, but plural names for the configuration
file.

To make the daemon configuration know how to correlate
the flag with the corresponding configuration option,
`opt.NewNamedListOptsRef()` should be used instead of
`opt.NewListOptsRef()`.

Commit 6702ac5 attempted
to fix the daemon not corresponding the flag with the configuration
file option, but did so by changing the name of the flag
to plural.

This patch reverts that change, and uses `opt.NewNamedListOptsRef()`
instead.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit 6e7715d)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah added this to the 17.12.1 milestone Jan 30, 2018
cpuguy83 pushed a commit to cpuguy83/docker-ce that referenced this pull request Feb 7, 2018
…ation

Fix presentation of published "random" host ports
Upstream-commit: 772956b263bda4f8e6c2d6e08b5c99d927230bb1
Component: cli
Copy link
Copy Markdown
Contributor

@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 🐸

Copy link
Copy Markdown
Contributor

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

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

LGTM

@andrewhsu andrewhsu merged commit e9f1a35 into docker-archive:17.12 Feb 13, 2018
@thaJeztah thaJeztah deleted the 17.12-backport-fix-plural-singular-node-generic-resources branch February 13, 2018 17:19
docker-jenkins pushed a commit that referenced this pull request Oct 12, 2019
[19.03 backport] revert controller: Check if IPTables is enabled for arrangeUserFilterRule ENGCORE-1114
Upstream-commit: adfac697dc5b74998c6a5229f1e4e9b2e0406af4
Component: engine
docker-jenkins pushed a commit that referenced this pull request Oct 28, 2019
deb: Only recommend aufs-tools on amd64
Upstream-commit: 7d1f550
Component: packaging
silvin-lubecki pushed a commit to silvin-lubecki/docker-ce that referenced this pull request Feb 3, 2020
…ix-plural-singular-node-generic-resources

[17.12] backport fix plural singular node generic resources
akrasnov-drv pushed a commit to drivenets/docker-ce that referenced this pull request Apr 23, 2023
deb: Only recommend aufs-tools on amd64
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