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

Avoid iptables rules affecting bridge forwarding #474

Closed
wants to merge 1 commit into from

Conversation

yaocw2020
Copy link
Contributor

@yaocw2020 yaocw2020 commented Apr 15, 2023

Problem:

In the Harvester cluster whose management network has a VLAN ID, the VM will be unable to access the host port or node port with the host IP where the VM is running.

Solution:

Disable net.bridge.bridge-nf-call-iptables to avoid iptables rules affecting bridge forwarding.

Related Issue: harvester/harvester#3414

Test plan:

  • Spin up a Harvester whose management network has a VLAN ID
  • Create a VM with VLAN network whose VLAN ID is same with the management network.
  • Curl nodeIP:443 in the VM.

@bk201
Copy link
Member

bk201 commented Apr 17, 2023

@yaocw2020 I feel the PR is not for issue 3744. Could we create a new issue to track it? Thanks.

@yaocw2020
Copy link
Contributor Author

@yaocw2020 I feel the PR is not for issue 3744. Could we create a new issue to track it? Thanks.

Sorry, it's issue 3414. I correct it.

@guangbochen
Copy link
Contributor

the test plan for this issue is not solid, we need to at least validate both the host network and guest k8s networks since this is a system-level configuration and its impact could be large.

@yaocw2020 yaocw2020 force-pushed the disableBridgeNF branch 2 times, most recently from 619d685 to b8ffe46 Compare May 23, 2023 03:09
@yaocw2020
Copy link
Contributor Author

Close because it could not solve the problem and I submit a new PR harvester/network-controller-harvester#83

@yaocw2020 yaocw2020 closed this May 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants