[ovs-dev] [PATCH ovn 4/4] ovn-northd: Support ECMP routes.

0-day Robot robot at bytheb.org
Tue Jan 7 03:17:52 UTC 2020


Bleep bloop.  Greetings Han Zhou, 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 94 characters long (recommended limit is 79)
#126 FILE: northd/ovn-northd.8.xml:2463:
          <code>reg8[0..15] == <var>GID</var> && reg8[16..31] == <var>MID</var></code>

WARNING: Line lacks whitespace around operator
WARNING: Line lacks whitespace around operator
WARNING: Line lacks whitespace around operator
#1164 FILE: utilities/ovn-nbctl.c:683:
  [--policy=POLICY] [--ecmp] lr-route-add ROUTER PREFIX NEXTHOP [PORT]\n\

WARNING: Line lacks whitespace around operator
WARNING: Line lacks whitespace around operator
WARNING: Line lacks whitespace around operator
#1167 FILE: utilities/ovn-nbctl.c:685:
  [--policy=POLICY] lr-route-del ROUTER [PREFIX [NEXTHOP [PORT]]]\n\

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