Re: [ovs-dev] [PATCH ovn] tests: Don't run ovn-northd specific tests with ovn-controller flavors.

2022-10-20 Thread Dumitru Ceara
On 10/18/22 21:55, Han Zhou wrote: > Acked-by: Han Zhou Thanks for the reviews, Mark and Numan! I pushed this change to the main branch. ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH ovn] tests: Don't run ovn-northd specific tests with ovn-controller flavors.

2022-10-18 Thread Han Zhou
On Wed, Oct 5, 2022 at 2:53 AM Dumitru Ceara wrote: > > There's no point to do that because ovn-controller is not even running > in ovn-northd.at tests. Wrap all tests in ovn-northd.at with calls to > the newly added OVN_FOR_EACH_NORTHD_NO_HV(). > > This also fixes the fact that some tests were n

Re: [ovs-dev] [PATCH ovn] tests: Don't run ovn-northd specific tests with ovn-controller flavors.

2022-10-18 Thread Mark Michelson
Thanks for the change, Dumitru. Acked-by: Mark Michelson On 10/5/22 05:53, Dumitru Ceara wrote: There's no point to do that because ovn-controller is not even running in ovn-northd.at tests. Wrap all tests in ovn-northd.at with calls to the newly added OVN_FOR_EACH_NORTHD_NO_HV(). This also

[ovs-dev] [PATCH ovn] tests: Don't run ovn-northd specific tests with ovn-controller flavors.

2022-10-05 Thread Dumitru Ceara
There's no point to do that because ovn-controller is not even running in ovn-northd.at tests. Wrap all tests in ovn-northd.at with calls to the newly added OVN_FOR_EACH_NORTHD_NO_HV(). This also fixes the fact that some tests were not run for all flavors of northd. Signed-off-by: Dumitru Ceara