Bleep bloop.  Greetings Lorenzo Bianconi, 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: The subject, '<area>: <summary>', is over 70 characters, i.e., 79.
Subject: controller: Update physical flows for peer port when the patch port is 
removed.
WARNING: Use of hardcoded table=<NUMBER> or resubmit=(,<NUMBER>) is discouraged 
in tests. Consider using MACRO instead.
#82 FILE: tests/ovn.at:40786:
AT_CHECK([ovs-ofctl dump-flows br-int table=65 | grep metadata=0x${ls_tnl_key} 
| grep -q load:0x2a->NXM_NX_REG14])

WARNING: Use of hardcoded table=<NUMBER> or resubmit=(,<NUMBER>) is discouraged 
in tests. Consider using MACRO instead.
#90 FILE: tests/ovn.at:40794:
AT_CHECK([ovs-ofctl dump-flows br-int table=65 | grep metadata=0x${ls_tnl_key} 
| grep -q load:0x2b->NXM_NX_REG14])

Lines checked: 98, Warnings: 3, 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