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

Add genev_sys_6081 to cilium devices #4575

Merged
merged 3 commits into from
Oct 8, 2024

Conversation

kvaps
Copy link
Contributor

@kvaps kvaps commented Oct 1, 2024

Signed-off-by: Andrei Kvapil kvapss@gmail.com

Pull Request

What type of this PR

Examples of user facing changes:

  • Bug fixes

I use kube-ovn with cilium, metallb and kubevirt.
I run ubuntu vm and expose 22 port with loadbalancer service
I can not access vm via externalIP assigned by metallb (externalTrafficPolicy: local)

Which issue(s) this PR fixes

It seems cilium is not correctly handled response packets recieved on genev_sys_6081 interface:

/ # tcpdump -nn -i any port 22
tcpdump: data link type LINUX_SLL2
tcpdump: verbose output suppressed, use -v[v]... for full protocol decode
listening on any, link-type LINUX_SLL2 (Linux cooked v2), snapshot length 262144 bytes
10:41:10.865457 eno5  In  IP 172.16.2.173.52314 > 10.2.0.242.22: Flags [S], seq 720481636, win 65535, options [mss 1410,nop,wscale 6,nop,nop,TS val 135386350 ecr 0,sackOK,eol], length 0
10:41:10.865465 bond0 In  IP 172.16.2.173.52314 > 10.2.0.242.22: Flags [S], seq 720481636, win 65535, options [mss 1410,nop,wscale 6,nop,nop,TS val 135386350 ecr 0,sackOK,eol], length 0
10:41:10.865537 ovn0  Out IP 172.16.2.173.52314 > 10.244.2.5.22: Flags [S], seq 720481636, win 65535, options [mss 1410,nop,wscale 6,nop,nop,TS val 135386350 ecr 0,sackOK,eol], length 0
10:41:10.865867 a33c4744248c_h Out IP 172.16.2.173.52314 > 10.244.2.5.22: Flags [S], seq 720481636, win 65535, options [mss 1410,nop,wscale 6,nop,nop,TS val 135386350 ecr 0,sackOK,eol], length 0
10:41:10.866210 a33c4744248c_h P   IP 10.244.2.5.22 > 172.16.2.173.52314: Flags [S.], seq 3405953119, ack 720481637, win 64704, options [mss 1360,sackOK,TS val 1687566854 ecr 135386350,nop,wscale 7], length 0
10:41:10.866270 genev_sys_6081 Out IP 10.244.2.5.22 > 172.16.2.173.52314: Flags [S.], seq 3405953119, ack 720481637, win 64704, options [mss 1360,sackOK,TS val 1687566854 ecr 135386350,nop,wscale 7], length 0

So I specidied it explicitly in cilium configuration, now packets are going fine

Signed-off-by: Andrei Kvapil <kvapss@gmail.com>
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Oct 1, 2024
kvaps added a commit to aenix-io/cozystack that referenced this pull request Oct 1, 2024
This PR includes the upstream fix:
- kubeovn/kube-ovn#4575

Signed-off-by: Andrei Kvapil <kvapss@gmail.com>
Makefile Outdated Show resolved Hide resolved
Co-authored-by: 张祖建 <zhangzujian.7@gmail.com>
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 8, 2024
@zhangzujian zhangzujian merged commit edd2ccf into kubeovn:master Oct 8, 2024
59 of 61 checks passed
zhangzujian added a commit that referenced this pull request Oct 8, 2024
* Add genev_sys_6081 to cilium devices

Signed-off-by: Andrei Kvapil <kvapss@gmail.com>

* Add vzlan_sys_4789 to cilium devices

Co-authored-by: 张祖建 <zhangzujian.7@gmail.com>

---------

Signed-off-by: Andrei Kvapil <kvapss@gmail.com>
Co-authored-by: 张祖建 <zhangzujian.7@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer need backport size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants