[ovs-dev] [PATCH ovn v9 3/4] northd: Add options to automatically add routes for NATs and LBs.

0-day Robot robot at bytheb.org
Thu Jul 1 00:25:02 UTC 2021


Bleep bloop.  Greetings Mark Michelson, 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 84 characters long (recommended limit is 79)
#162 FILE: northd/ovn-northd.c:3637:
            bool is_routable = smap_get_bool(&lb->nlb->options, "add_route", false);

WARNING: Line has trailing whitespace
#297 FILE: ovn-nb.xml:2078:
          load balancer IP addresses to the appropriate MAC address. Setting 

WARNING: Line lacks whitespace around operator
#425 FILE: utilities/ovn-nbctl.c:370:
  [--add-route]\n\

Lines checked: 508, Warnings: 3, Errors: 0


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