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

Bridged packet matching VLAN broadcast entry even when unicast entry exists #69

Closed
ccascone opened this issue Sep 16, 2020 · 4 comments
Closed
Labels
bug Something isn't working

Comments

@ccascone
Copy link
Member

I'm sending the following packet:

21:24:22.548175 00:1e:67:d2:ee:ea > 00:1e:67:d2:ce:52, ethertype IPv4 (0x0800), length 98: (tos 0x0, ttl 64, id 54276, offset 0, flags [DF], proto ICMP (1), length 84)
    192.168.250.20 > 192.168.250.21: ICMP echo request, id 8766, seq 62, length 64

But ONOS reports that the VLAN broadcast entry is the one being matched, even when there is one for dest MAC 00:1e:67:d2:ce:52:
Screen Shot 2020-09-16 at 2 25 25 PM

@ccascone ccascone added the bug Something isn't working label Sep 16, 2020
@ccascone
Copy link
Member Author

To reproduce, deploy ONOS and Stratum on menlo-prd using patchset 37 from:
https://gerrit.opencord.org/c/aether-pod-configs/+/19963

Start a ping from node edge1 to edge2:

cord@demo-edge-01:~$ sudo ping -i0.1 192.168.250.21

@charlesmcchan
Copy link
Collaborator

@ccascone Were you seeing a traffic disruption when this issue is observed? Even if the packet get broadcasted, it should still reach the destination, correct?

@ccascone
Copy link
Member Author

ccascone commented Sep 16, 2020

That's correct, ping works, but packets are broadcasted to all other ports on VLAN 250

@ccascone
Copy link
Member Author

This was an issue with stratum-bfrt not handling priority according to the p4rt spec. Fixed in stratum/stratum@03eb8f0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants