[ovs-dev] [PATCH v9] ovn: Add a new logical switch port type - 'virtual'

0-day Robot robot at bytheb.org
Thu Jul 18 15:58:47 UTC 2019


Bleep bloop.  Greetings Numan Siddique, 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:
ERROR: Co-author Ben Pfaff <blp at ovn.org> needs to sign off.
WARNING: Line is 236 characters long (recommended limit is 79)
#511 FILE: ovn/northd/ovn-northd.8.xml:530:
<code>inport == <var>P</var> &amp;&amp; !is_chassis_resident(<var>V</var>) &amp;&amp; ((arp.op == 1 &amp;&amp; arp.spa == <var>VIP</var> &amp;&amp; arp.tpa == <var>VIP</var>) || (arp.op == 2 &amp;&amp; arp.spa == <var>VIP</var>))</code>

Lines checked: 1435, Warnings: 1, Errors: 1


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

Thanks,
0-day Robot


More information about the dev mailing list