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

[18.09] backport default-addr-pool-mask-length param max value check #73

Merged
merged 2 commits into from
Oct 11, 2018
Merged

[18.09] backport default-addr-pool-mask-length param max value check #73

merged 2 commits into from
Oct 11, 2018

Commits on Oct 4, 2018

  1. Global Default AddressPool - Update 

    Addressing few review comments as part of code refactoring.
Also moved validation logic from CLI to Moby.

Signed-off-by: selansen <elango.siva@docker.com>
(cherry picked from commit 148ff00)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
    @selansen @thaJeztah
    selansen authored and thaJeztah committed Oct 4, 2018
    Configuration menu
    Copy the full SHA
    9816bfc View commit details
    Browse the repository at this point in the history

  2. Fix for default-addr-pool-mask-length param max value check 

    We check for max value for -default-addr-pool-mask-length param as 32.
But There won't be enough addresses on the  overlay network. Hence we are
keeping it 29 so that we would be having atleast 8 addresses in /29 network.

Signed-off-by: selansen <elango.siva@docker.com>
(cherry picked from commit d25c5df)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
    @selansen @thaJeztah
    selansen authored and thaJeztah committed Oct 4, 2018
    Configuration menu
    Copy the full SHA
    9406f36 View commit details
    Browse the repository at this point in the history