Re: [ovs-dev] [PATCH ovn 2/2] Omit ACLs for nd || nd_ra || nd_rs || mldv1 || mldv2

2023-04-13 Thread Ihar Hrachyshka
I'm not sure but maybe it's something to do with the difference between python3 paths in github CI and locally in my env? I see python3 in github is /opt/hostedtoolcache/Python/3.11.2/x64/bin/python3. I will try $PYTHON3 in the function definition and see if it works better. On Thu, Apr 13, 2023

Re: [ovs-dev] [PATCH ovn 2/2] Omit ACLs for nd || nd_ra || nd_rs || mldv1 || mldv2

2023-04-13 Thread Dumitru Ceara
On 4/13/23 10:28, Dumitru Ceara wrote: > One is because the tests fail to run scapy: > > Traceback (most recent call last): > File "", line 1, in > ModuleNotFoundError: No module named 'scapy' > "netdev-dummy/receive" command requires at least 2 arguments > ovs-appctl: ovs-vswitchd: server

Re: [ovs-dev] [PATCH ovn 2/2] Omit ACLs for nd || nd_ra || nd_rs || mldv1 || mldv2

2023-04-13 Thread Dumitru Ceara
On 4/12/23 03:29, Ihar Hrachyshka wrote: > When setting flows for LS, OVN distinguishes between two states: where > there’s a stateful ACL present in its list (has_stateful == true *) > and when it’s missing (all ACLs are stateless). > > When has_stateful == true, the following is done (among

[ovs-dev] [PATCH ovn 2/2] Omit ACLs for nd || nd_ra || nd_rs || mldv1 || mldv2

2023-04-11 Thread Ihar Hrachyshka
When setting flows for LS, OVN distinguishes between two states: where there’s a stateful ACL present in its list (has_stateful == true *) and when it’s missing (all ACLs are stateless). When has_stateful == true, the following is done (among other things): - ct handling flows are set; - they are