Comment 15 for bug 1531013

Revision history for this message
Daniel (dlevy) wrote :

@Darragh
The patch as it was did not work.
I believe it had to do with the way you were comparing the variable 'entry'. You defined it like this: '%s dst %s self permanent' wheres entries could take a different form (ours look like this '%s dev vxlan-2 dst %s self permanent'), so I parsed it to extract the comparision parameters (mac and ip) and just compare those.

I was actually thinking that if there is not an exact match it should be added (both mac and ip). I just added line 621 to align with your code and the original code, but not sure if needed.