Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update getting-started/quickstart demo network to use the firewall CNI plugin #265

Closed
sipsma opened this issue Sep 24, 2019 · 0 comments · Fixed by #281
Closed

Update getting-started/quickstart demo network to use the firewall CNI plugin #265

sipsma opened this issue Sep 24, 2019 · 0 comments · Fixed by #281
Assignees
Labels
documentation Issue related to documentation

Comments

@sipsma
Copy link
Contributor

sipsma commented Sep 24, 2019

We need a fix to the firewall CNI plugin, which I've submitted a PR for here, but once that's merged we should update the demo network to use it as it results in more consistent success in setting up forwarding from VMs to all networks the host has access to (relative to the ptp plugin's ipmasq attempts, which seem hit or miss especially if the host already has iptables rules configured).

My previous test with a patched firewall plugin found that simply appending it after ptp but before tc-redirect-tap in the existing demo network conflist resulted in VMs consistently getting outbound network access where previously they did not.

@sipsma sipsma self-assigned this Sep 24, 2019
@samuelkarp samuelkarp added the documentation Issue related to documentation label Sep 24, 2019
fangn2 pushed a commit to fangn2/firecracker-containerd that referenced this issue Mar 23, 2023
…ependabot/go_modules/github.com/sirupsen/logrus-1.7.0

Bump github.com/sirupsen/logrus from 1.6.0 to 1.7.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Issue related to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants