On Mon, 1 Dec 2025 at 06:49, Mike Pattrick <[email protected]> wrote: >> +dnl Simple ARP. >> +$PYTHON3 -c dnl >> +"from scapy.all import Ether, ARP; "dnl >> +"p = Ether(dst='ff:ff:ff:ff:ff:ff'); "dnl >> +"p /= ARP(pdst='192.168.123.1', hwsrc='8a:bf:7e:2f:05:84'); "dnl >> +"print(p.build().hex())" > arp_frame > > > I think the code changes look reasonable, but is it acceptable to include > scapy as a dependency in the build tests? It appears to only be currently > used in the two MFEX autovalidator tests, and isn't listed on the build > requirements page.
Ilya also expressed concerns about scapy invocation time. I have a v3 series, dropping use of scapy, and using fixed hexadecimal strings (but with comments on how those were generated). -- David Marchand _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
