On Sun, Oct 2, 2016 at 11:43 AM, Han Zhou <zhou...@gmail.com> wrote:

> This change should belong to the 1/3 patch. Each individual patch should
> be complete and independent to future patches.
>

This part is an RFC, so I was trying to make it easier to digest.
Normally, this would be part of patch 1, for the reason patch 1 breaks this
test -  that is one rule
we use.



>
> On Sat, Oct 1, 2016 at 4:34 PM, Darrell Ball <dlu...@gmail.com> wrote:
>
>> If arp responders are unnecessay for logical switch
>> "router type" ports. then an adjustment is necessary
>> for a test.
>>
>> Signed-off-by: Darrell Ball <dlu...@gmail.com>
>> ---
>>  tests/ovn.at | 4 +---
>>  1 file changed, 1 insertion(+), 3 deletions(-)
>>
>> diff --git a/tests/ovn.at b/tests/ovn.at
>> index f40940f..aff1a39 100644
>> --- a/tests/ovn.at
>> +++ b/tests/ovn.at
>> @@ -2208,9 +2208,7 @@ test_arp() {
>>      local j k
>>      for j in 1 2 3; do
>>          for k in 1 2 3; do
>> -            # 192.168.33.254 is configured to the switch patch port for
>> lrp33,
>> -            # so no ARP flooding expected for it.
>> -            if test $i$j$k != $inport && test $tpa != `ip_to_hex 192 168
>> 33 254`; then
>> +            if test $i$j$k != $inport; then
>>                  echo $request >> $i$j$k.expected
>>              fi
>>          done
>> --
>> 1.9.1
>>
>> _______________________________________________
>> dev mailing list
>> dev@openvswitch.org
>> http://openvswitch.org/mailman/listinfo/dev
>>
>
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to