Skip to content

[27.x backport] dockerd: fix docs, improve validation and improve coverage of "--feature" flag#48503

Merged
thaJeztah merged 5 commits intomoby:27.xfrom
thaJeztah:27.x_backport_dockerd_typo
Sep 16, 2024
Merged

[27.x backport] dockerd: fix docs, improve validation and improve coverage of "--feature" flag#48503
thaJeztah merged 5 commits intomoby:27.xfrom
thaJeztah:27.x_backport_dockerd_typo

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

man: fix duplicate word in --feature flag description

man: dockerd: value is optional for --feature flag

The --feature flag allows the boolean value to be omitted. If only a name is provided, the default is "true".

internal/opts: SetOpts,NamedSetOpts: test for optional value

The value is optional for SetOpts (and NamedSetOpts), and implied "true" when omitted.

This patch adds a test-case for this.

internal/opts: SetOpts.Set: remove redundant var assignment

internal/opts: SetOpts: invalidate empty option-names

Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit 2b6550b)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
The --feature flag allows the boolean value to be omitted.
If only a name is provided, the default is "true".

Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit 50e83a0)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
The value is optional for SetOpts (and NamedSetOpts), and implied
"true" when omitted.

This patch adds a test-case for this.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit 758cca6)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit 908bb95)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit c2fc1f4)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 27.3.0 milestone Sep 16, 2024
@thaJeztah thaJeztah self-assigned this Sep 16, 2024
@thaJeztah thaJeztah marked this pull request as ready for review September 16, 2024 11:52
@thaJeztah thaJeztah requested a review from laurazard September 16, 2024 11:52
@thaJeztah thaJeztah merged commit aed88ad into moby:27.x Sep 16, 2024
@thaJeztah thaJeztah deleted the 27.x_backport_dockerd_typo branch September 16, 2024 12:12
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.

2 participants