Bleep bloop.  Greetings Xavier Simonart, I am a robot and I have tried out your 
patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
WARNING: Use of hardcoded table=<NUMBER> or resubmit=(,<NUMBER>) is discouraged 
in tests. Consider using MACRO instead.
#473 FILE: tests/ovn.at:16606:
  AT_CHECK_UNQUOTED([as $hv ovs-ofctl dump-flows br-int table=0 | grep 
"in_port=$of" | wc -l], [0], [dnl

WARNING: Use of hardcoded table=<NUMBER> or resubmit=(,<NUMBER>) is discouraged 
in tests. Consider using MACRO instead.
#493 FILE: tests/ovn.at:16626:
        test 1 -le $(as hv1 ovs-ofctl dump-flows br-int table=0 | grep 
"in_port=$of_hv1" | wc -l) &&

WARNING: Use of hardcoded table=<NUMBER> or resubmit=(,<NUMBER>) is discouraged 
in tests. Consider using MACRO instead.
#494 FILE: tests/ovn.at:16627:
        test 1 -le $(as hv2 ovs-ofctl dump-flows br-int table=0 | grep 
"in_port=$of_hv2" | wc -l)

WARNING: Use of hardcoded table=<NUMBER> or resubmit=(,<NUMBER>) is discouraged 
in tests. Consider using MACRO instead.
#599 FILE: tests/ovn.at:38614:
OVS_WAIT_FOR_OUTPUT([as hv1 ovs-ofctl dump-flows br-int table=0 |grep 
priority=150|grep dl_vlan=7| grep -c in_port=8], [0],[dnl

Lines checked: 607, Warnings: 4, Errors: 0


Please check this out.  If you feel there has been an error, please email 
[email protected]

Thanks,
0-day Robot
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to