On 3/21/25 3:18 PM, Xavier Simonart wrote: > Hi Dumitru > Hi Xavier,
> Thanks for the patch. > It looks good to me, thanks for fixing this! > Well, sorry that I broke it in the first place! :) > Acked-by: Xavier Simonart <[email protected]> Thanks a lot, applied to main! Regards, Dumitru > Thanks > Xavier > > On Fri, Mar 21, 2025 at 3:16 PM Dumitru Ceara <[email protected]> wrote: > >> Commit e64a15ef7715 ("Modified the CI to use ovn-fake-multinode v0.3.") >> changed the ovn-fake-multinode version that's used by our multinode >> tests. In the new version default container names are slightly >> different. When I applied 3baa8c6421ab ("multinode tests: >> Simplify/Cleanup migration test.") I didn't take that into account >> during the rebase. >> >> This commit fixes the multinode_sbctl() implementation. >> >> Fixes: 3baa8c6421ab ("multinode tests: Simplify/Cleanup migration test.") >> Signed-off-by: Dumitru Ceara <[email protected]> >> --- >> tests/multinode-macros.at | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/tests/multinode-macros.at b/tests/multinode-macros.at >> index d14fafcb5c..cede802a95 100644 >> --- a/tests/multinode-macros.at >> +++ b/tests/multinode-macros.at >> @@ -94,7 +94,7 @@ multinode_nbctl () { >> } >> >> multinode_sbctl () { >> - m_as ovn-central-az1-1 ovn-sbctl "$@" >> + m_as ovn-central-az1 ovn-sbctl "$@" >> } >> >> check_fake_multinode_setup_by_nodes() { >> -- >> 2.48.1 >> >> > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
