[ovs-dev] [PATCH ovn 12/12] ovn-northd-ddlog: New implementation of ovn-northd based on ddlog.

0-day Robot robot at bytheb.org
Wed Nov 4 08:13:49 UTC 2020


Bleep bloop.  Greetings Ben Pfaff, 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: Comment with 'xxx' marker
#4578 FILE: northd/ovn-northd-ddlog.c:1273:
     * XXX If the transaction we're sending to the database fails, then

WARNING: Line lacks whitespace around operator
WARNING: Line lacks whitespace around operator
WARNING: Line lacks whitespace around operator
#4782 FILE: northd/ovn-northd-ddlog.c:1477:
  --ovnnb-db=DATABASE       connect to ovn-nb database at DATABASE\n\

WARNING: Line lacks whitespace around operator
WARNING: Line lacks whitespace around operator
WARNING: Line lacks whitespace around operator
#4784 FILE: northd/ovn-northd-ddlog.c:1479:
  --ovnsb-db=DATABASE       connect to ovn-sb database at DATABASE\n\

WARNING: Line lacks whitespace around operator
WARNING: Line lacks whitespace around operator
#4786 FILE: northd/ovn-northd-ddlog.c:1481:
  --unixctl=SOCKET          override default control socket name\n\

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