frr icon indicating copy to clipboard operation
frr copied to clipboard

zebra: correct one comment for evpn-mh

Open anlancs opened this issue 2 years ago • 2 comments

These moved mac addresses are actually in active status during moving phase, just correct comment.

anlancs avatar Aug 09 '22 05:08 anlancs

Continuous Integration Result: SUCCESSFUL

Congratulations, this patch passed basic tests

Tested-by: NetDEF / OpenSourceRouting.org CI System

CI System Testrun URL: https://ci1.netdef.org/browse/FRR-PULLREQ2-6834/

This is a comment from an automated CI system. For questions and feedback in regards to this CI system, please feel free to email Martin Winter - mwinter (at) opensourcerouting.org.

NetDEF-CI avatar Aug 09 '22 07:08 NetDEF-CI

@chiragshah6 -> is this comment change appropriate?

donaldsharp avatar Aug 09 '22 11:08 donaldsharp

It would be good, in your PRs you provide minimum/relevant UT case. simulate moves and capture following traces:

Got. Just append the trace in this moving case:

2022/07/29 01:27:27 ZEBRA: [WW9R1-2QR27] dpAdd local-nw-MAC e4:f0:04:89:b6:46 VID 30                         
2022/07/29 01:27:27 ZEBRA: [TVHPV-4R7K8] UPD MAC e4:f0:04:89:b6:46 intf bond1(18) VID 30 -> VNI 10030 curFlags AUTO LOC Active        
2022/07/29 01:27:27 ZEBRA: [V6Q00-48BR2] local mac vni 10030 mac e4:f0:04:89:b6:46 es 03:00:22:22:22:22:22:00:00:01 seq 0 f LOC LOC Active 

anlancs avatar Aug 13 '22 14:08 anlancs