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

[WIP] Stabilize CNI connectivity, dns and latent #421

Closed
wants to merge 3 commits into from

Conversation

chanwit
Copy link
Member

@chanwit chanwit commented Sep 11, 2019

This PR fixes a containerd + CNI behavior that users cannot connect to the internet.
After investigation, I have found that there's not FORWARD rules to help the CNI bridge plugin to complete this. So I add simple rules for the Ignite network CIDR to allow forwarding.

fixes #418 #423 #424

@chanwit chanwit added kind/bug Categorizes issue or PR as related to a bug. area/networking Issues related to networking labels Sep 11, 2019
@chanwit chanwit added this to the v0.6.1 milestone Sep 11, 2019
@chanwit
Copy link
Member Author

chanwit commented Sep 11, 2019

How to test:

$ sudo rm /etc/cni/net.d/10-ignite.conflist 
$ sudo ifconfig cni0 down
$ sudo ip link delete cni0
$ sudo bin/ignite run weaveworks/ignite-ubuntu --ssh
INFO[0006] Created VM with ID "17b100016889f1d3" and name "white-wave" 
INFO[0007] Networking is handled by "cni"               
INFO[0007] Started Firecracker VM "17b100016889f1d3" in a container with ID "ignite-17b100016889f1d3" 
$ sudo bin/ignite ps
VM ID                   IMAGE                           KERNEL                                  SIZE    CPUS    MEMORY          CREATED STATUS  IPS             PORTS   NAME
17b100016889f1d3        weaveworks/ignite-ubuntu:latest weaveworks/ignite-kernel:4.19.47        4.0 GB  1       512.0 MB        6s ago  Up 6s   172.18.0.4              white-wave
$ sudo bin/ignite ssh 17b

root@17b100016889f1d3:~# ping 8.8.8.8
PING 8.8.8.8 (8.8.8.8) 56(84) bytes of data.
64 bytes from 8.8.8.8: icmp_seq=1 ttl=47 time=36.4 ms
64 bytes from 8.8.8.8: icmp_seq=2 ttl=47 time=36.2 ms
64 bytes from 8.8.8.8: icmp_seq=3 ttl=47 time=36.3 ms
^C
--- 8.8.8.8 ping statistics ---
3 packets transmitted, 3 received, 0% packet loss, time 2003ms
rtt min/avg/max/mdev = 36.255/36.344/36.428/0.070 ms
root@17b100016889f1d3:~# 

@chanwit
Copy link
Member Author

chanwit commented Sep 11, 2019

@stealthybox PTAL

@chanwit
Copy link
Member Author

chanwit commented Sep 12, 2019

This PR needs rebase on top of #422

Also CNI does not have the managed DNS like Docker libnetwork,
so this --dns could provide name services for VM when needed.
@chanwit chanwit changed the title Add forward rules for the default CIDR Stabilize CNI connectivity, dns and latent Sep 12, 2019
@chanwit
Copy link
Member Author

chanwit commented Sep 12, 2019

Test cases implemented by #422 used to verify the behavior and all tests passed.

@chanwit chanwit changed the title Stabilize CNI connectivity, dns and latent [WIP] Stabilize CNI connectivity, dns and latent Sep 13, 2019
@chanwit chanwit removed this from the v0.6.1 milestone Sep 14, 2019
@chanwit
Copy link
Member Author

chanwit commented Sep 15, 2019

withdrawn in favor of #427 #428 #429

@chanwit chanwit closed this Sep 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/networking Issues related to networking kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

When using containerd + CNI network, the VM could not ping the Internet IPs
1 participant