[ovs-dev] [PATCH v4 ovn 1/2] Forwarding group to load balance l2 traffic with liveness detection

0-day Robot robot at bytheb.org
Wed Jan 22 21:58:35 UTC 2020


Bleep bloop.  Greetings Manoj Sharma, 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: Line is 149 characters long (recommended limit is 79)
#420 FILE: utilities/ovn-nbctl.8.xml:489:
      <dt>[<code>--liveness</code>]<code>fwd-group-add</code> <var>group</var> <var>switch</var> <var>vip</var> <var>vmac</var> <var>ports</var></dt>

WARNING: Line lacks whitespace around operator
#468 FILE: utilities/ovn-nbctl.c:653:
  fwd-group-add GROUP SWITCH VIP VMAC PORTS...\n\

WARNING: Line lacks whitespace around operator
#470 FILE: utilities/ovn-nbctl.c:655:
  fwd-group-del GROUP       delete a forwarding group\n\

WARNING: Line lacks whitespace around operator
#471 FILE: utilities/ovn-nbctl.c:656:
  fwd-group-list [SWITCH]   print forwarding groups\n\

ERROR: Improper whitespace around control block
#492 FILE: utilities/ovn-nbctl.c:4843:
        NBREC_FORWARDING_GROUP_FOR_EACH(fwd_group, ctx->idl) {

ERROR: Improper whitespace around control block
#688 FILE: utilities/ovn-nbctl.c:5039:
    NBREC_FORWARDING_GROUP_FOR_EACH(fwd_group, ctx->idl) {

Lines checked: 739, Warnings: 4, Errors: 2


Please check this out.  If you feel there has been an error, please email aconole at redhat.com

Thanks,
0-day Robot


More information about the dev mailing list