Skip to content

[19.03 backport] Add interpolation type cast for max_replicas_per_node - #2184

Merged
thaJeztah merged 1 commit into
docker:19.03from
thaJeztah:19.03_backport_fix_max_replicas_per_node_interpolation
Jan 16, 2020
Merged

[19.03 backport] Add interpolation type cast for max_replicas_per_node#2184
thaJeztah merged 1 commit into
docker:19.03from
thaJeztah:19.03_backport_fix_max_replicas_per_node_interpolation

Conversation

@thaJeztah

@thaJeztahthaJeztah commented Nov 5, 2019

Copy link
Copy Markdown
Member

backport of #2182

Fixesdocker-archive-public/docker.app#688

- What I did
Added interpolation type cast mapping for missing deploy.placement.max_replicas_per_node.

- How to verify it

Changed the unit test

- A picture of a cute animal (not mandatory but encouraged)
image

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Fixesdocker-archive-public/docker.app#688
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
(cherry picked from commit cb29ef6)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah

Copy link
Copy Markdown
MemberAuthor

ping @rumpl@silvin-lubecki PTAL

@rumpl

rumpl commented Nov 6, 2019

Copy link
Copy Markdown
Member

👍

@thaJeztahthaJeztah added this to the 19.03.6 milestone Nov 8, 2019
@thaJeztah

Copy link
Copy Markdown
MemberAuthor

ping @silvin-lubecki@tiborvass PTAL

@thaJeztahthaJeztah changed the title [19.03 backport] Add interpolation type cast for max_replicas_per_node[WIP][19.03 backport] Add interpolation type cast for max_replicas_per_nodeDec 4, 2019
@thaJeztah

This comment has been minimized.

@thaJeztahthaJeztah changed the title [WIP][19.03 backport] Add interpolation type cast for max_replicas_per_node[19.03 backport] Add interpolation type cast for max_replicas_per_nodeJan 16, 2020
@thaJeztah

Copy link
Copy Markdown
MemberAuthor

LGTM

@thaJeztah
thaJeztah merged commit a7cad2f into docker:19.03Jan 16, 2020
@thaJeztah
thaJeztah deleted the 19.03_backport_fix_max_replicas_per_node_interpolation branch January 16, 2020 20:17
Sign up for freeto 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.

4 participants

@thaJeztah@rumpl@tonistiigi@GordonTheTurtle