On 7/13/23 18:01, Ihar Hrachyshka wrote:
> Reviewed-By: Ihar Hrachyshka <i...@redhat.com>
> 
> +1. Removing IHAR from OVN? Long overdue. :)
> 

I'm sure you know I didn't mean anything like that. I hope to see many
more contributions from IHAR in the future! :D

Thanks for the review, I pushed this to main and backported it to all
branches down to 23.03.

Regards,
Dumitru

> On Thu, Jul 13, 2023 at 11:30 AM Dumitru Ceara <dce...@redhat.com> wrote:
> 
>> Remove what seems to be an accidental test debug echo command.
>>
>> Fixes: ab7b0eb8ca05 ("Add connectivity test for 2 controllers on the same
>> host")
>> Signed-off-by: Dumitru Ceara <dce...@redhat.com>
>> ---
>>  tests/ovn-macros.at | 1 -
>>  1 file changed, 1 deletion(-)
>>
>> diff --git a/tests/ovn-macros.at b/tests/ovn-macros.at
>> index 6f2d085ae4..30478140e9 100644
>> --- a/tests/ovn-macros.at
>> +++ b/tests/ovn-macros.at
>> @@ -371,7 +371,6 @@ start_virtual_controller() {
>>          -- set bridge $int_bridge fail-mode=secure
>> other-config:disable-in-band=true \
>>          || return 1
>>
>> -    echo IHAR ${cli_args}
>>      ovn-controller --enable-dummy-vif-plug ${cli_args} -vconsole:off
>> --detach --no-chdir
>>  }
>>
>> --
>> 2.31.1
>>
>>
> 

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to