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

ensure vxlan mac will be correct with systemd-network MACAddressPolicy=random #1796

Merged
merged 1 commit into from
Sep 18, 2023

Conversation

zhangguanzhang
Copy link
Contributor

@zhangguanzhang zhangguanzhang commented Aug 21, 2023

Fixes: #1795

Description

after exit, flanneld will update vtep info:

$ docker logs ae5e1a274eae
E0821 11:40:16.145538       1 main.go:335] Error registering network: failed to configure interface flannel.1: flannel.1's mac address wanto be: 36:49:83:64:e0:11 but got: 7e:fa:35:d0:2a:d0
I0821 11:40:16.145771       1 main.go:523] Stopping shutdownHandler...
$ docker ps -a | grep flanneld
b89f5d8163e2   f18629ca5ecb                                         "/opt/bin/flanneld -…"   2 seconds ago    Up 1 second                             k8s_kube-flannel_kube-flannel-ds-9h5tc_kube-system_92426357-db8b-4f70-b536-f9de959d0881_7
ae5e1a274eae   f18629ca5ecb                                         "/opt/bin/flanneld -…"   23 seconds ago   Exited (1) 14 seconds ago               k8s_kube-flannel_kube-flannel-ds-9h5tc_kube-system_92426357-db8b-4f70-b536-f9de959d0881_6

Todos

  • Tests
  • Documentation
  • Release note

Release Note

None required

@zhangguanzhang
Copy link
Contributor Author

@rbrtbnfgl PTAL

@rbrtbnfgl rbrtbnfgl merged commit 8bb937c into flannel-io:master Sep 18, 2023
8 checks passed
@zhangguanzhang zhangguanzhang deleted the v0.22.1-test branch September 18, 2023 15:35
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.

wrong VtepMAC after flannel.1 first up
3 participants