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

Change default CNI network name, bridge name, and subnet #460

Merged
merged 1 commit into from
Sep 26, 2019

Conversation

stealthybox
Copy link
Contributor

@stealthybox stealthybox commented Sep 26, 2019

Fixes #424
Fixes #455

The solution in this patch for #424 is to use a static subnet in the 10.0.0.0/8 range.
This avoids the complexity of collaborating with the docker defaulting logic which requires calculating and iterating a over a similar list of subnets and checking to see if they're in use by the kernel.
This:

  • keeps the code simple
  • has the future side-effect of making issues more searchable via google when users submit issue details with the default network setup

Rationale for the subnet choice is documented on the constant.

Changing these CNI details will ultimately necessitate that any current users of our (mostly broken) default CNI setup will likely want to migrate.
This is not forced on users though. The config is still only written when the CNI net conf dir is empty.

@stealthybox stealthybox added this to the v0.6.1 milestone Sep 26, 2019
@stealthybox stealthybox added the area/networking Issues related to networking label Sep 26, 2019
@stealthybox
Copy link
Contributor Author

cc @najeal -- pinging you as discussed
Please review at your leisure :)

Copy link
Member

@chanwit chanwit left a comment

Choose a reason for hiding this comment

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

both changes

  • ignite0
    and
  • 10.61.0.0/16
    lgtm

Copy link
Contributor

@najeal najeal left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

@chanwit
Copy link
Member

chanwit commented Sep 26, 2019

Thank you @najeal to help review this, and thank you @stealthybox for taking care of this.

@lucidprogrammer
Copy link

Thanks for this update, however, its possibly causing #462

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/networking Issues related to networking
Projects
None yet
4 participants