-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
rawhide: MAC address changed after podman network reload #9720
Comments
Can you add |
I'll keep this VM up for a few hours, say until 1200 EDT / 1700 CET, please ping me here or on Freenode if there's anything else you need me to test. |
Forgot to mention: containernetworking-plugins-0.9.1-9.1.git2989aba.fc35.x86_64 |
Looks like something on the cni side is not working correctly. |
Oh, good idea: # podman exec cid ip link show ! before
3: eth0@if27: <BROADCAST,MULTICAST,UP,LOWER_UP,M-DOWN> mtu 1500 qdisc noqueue state UP
link/ether 62:48:46:79:a5:77 brd ff:ff:ff:ff:ff:ff
# podman network reload cid
# podman exec cid ip link show ! after
5: eth0@if4: <BROADCAST,MULTICAST,UP,LOWER_UP,M-DOWN> mtu 1500 qdisc noqueue state UP
link/ether 62:48:46:79:a5:77 brd ff:ff:ff:ff:ff:ff
# podman inspect 0ec51ce24e0efb1a89253d68adbf033da4da4de139b93921fec280b78813bc8c --format {{.NetworkSettings.MacAddress}}
7a:d1:69:bd:ed:f8
#/vvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvv
#| FAIL: MAC address changed after podman network reload
#| expected: '62:48:46:79:a5:77'
#| actual: '7a:d1:69:bd:ed:f8'
#\^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ |
When did this start failing? And fails it every time or is it a flake? |
Sorry, forgot to reply to that part: it seems to be 100% failure, i.e. not a flake. I don't know when it started failing because we have another failure (#9507) that made me stop bothering to look at gating tests until it was fixed. I don't think the #9507 failure included this MAC address error, but unfortunately the logs have disappeared. The MAC address error was there last Wednesday when I looked, but I was on PTO. So, let's say some time between Feb 24 and Mar 10. |
OK I just compiled the cni plugins from master and can reproduce. I will bisect and report the error there. |
@edsantiago That's my test :) |
Oops! I blame |
A friendly reminder that this issue had no activity for 30 days. |
It seems Has been fixed
|
@zhangguanzhang You have to use the cni plugins compiled from master. This still reproduces with the latest changes. |
Not fixed. Log from last night's podman-3.2.0-0.1.dev.git373f15f.fc35 build. |
Not sure what was just said. @Luap99 are you saying it is fixed in the latest cni plugins, or broken in the latest cni plugins? |
It is broken with the latest builds from master but works fine with the latest release 0.9.1. |
This should be fixed upstream containernetworking/plugins#626 |
podman gating tests regularly failing on rawhide with:
The text was updated successfully, but these errors were encountered: