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

ANP with FQDN rules does not work correctly on TCP traffic #5077

Closed
wenyingd opened this issue Jun 5, 2023 · 0 comments · Fixed by #5078
Closed

ANP with FQDN rules does not work correctly on TCP traffic #5077

wenyingd opened this issue Jun 5, 2023 · 0 comments · Fixed by #5078
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@wenyingd
Copy link
Contributor

wenyingd commented Jun 5, 2023

Describe the bug

In some cases, TCP port 53 is also used for DNS protocol. When configuring with ANP rules on FQDN, we expected to send the DNS response to antrea-agent no matter it is using UDP or TCP protocol. When it is using TCP protocol, the relevant OpenFlow flow is supposed to be something like,

cookie=0x1020000000000, table=AntreaPolicyIngressRule, priority=64991,tcp,tp_src=53,tcp_flags=+psh+ack actions=conjunction(1,1/2)

However, the real flow installed on OVS is like this,

cookie=0x47020000000000, duration=261903.244s, table=AntreaPolicyIngressRule, n_packets=0, n_bytes=0, idle_age=65535, priority=64991,tcp,tcp_flags=+psh+ack actions=conjunction(1,1/2)

It means that all TCP packets marked wth "ack" and "push" packet sending to the "appliedTo" Pod would be sent to antrea-agent via the PacketIn mechnism.

To Reproduce

Install antrea v1.12, and create an ANP with fqdn rules. Then dump OpenFlow entries on antrea-agent Pod.

Expected

Only TCP DNS response ( tp_src=53 and tcp_flags=+ack+psh ) would be sent to antrea-agent when FQDN rules are applied.

Actual behavior

After ANP with FQDN rules are applied, all TCP packets marked with ack and psh which are sending to the appliedTo Pods are sent to antrea-agent.

Versions:

Antrea v1.12

Additional context

@wenyingd wenyingd added the kind/bug Categorizes issue or PR as related to a bug. label Jun 5, 2023
@wenyingd wenyingd mentioned this issue Jun 6, 2023
@luolanzone luolanzone linked a pull request Jun 7, 2023 that will close this issue
@tnqn tnqn closed this as completed in #5078 Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant