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 82 characters long (recommended limit is 79) #125 FILE: ic/ovn-ic.8.xml:54: <xi:include href="lib/daemon.xml" xmlns:xi="http://www.w3.org/2003/XInclude"/> WARNING: Line is 80 characters long (recommended limit is 79) #128 FILE: ic/ovn-ic.8.xml:57: <xi:include href="lib/vlog.xml" xmlns:xi="http://www.w3.org/2003/XInclude"/> WARNING: Line lacks whitespace around operator WARNING: Line lacks whitespace around operator WARNING: Line lacks whitespace around operator #266 FILE: ic/ovn-ic.c:78: --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 #268 FILE: ic/ovn-ic.c:80: --ovnsb-db=DATABASE connect to ovn-sb database at DATABASE\n\ WARNING: Line lacks whitespace around operator WARNING: Line lacks whitespace around operator #270 FILE: ic/ovn-ic.c:82: --unixctl=SOCKET override default control socket name\n\ WARNING: Comment with 'xxx' marker #485 FILE: ic/ovn-ic.c:297: /* ovn-nb db. XXX: add only needed tables and columns */ WARNING: Comment with 'xxx' marker #489 FILE: ic/ovn-ic.c:301: /* ovn-sb db. XXX: add only needed tables and columns */ WARNING: Line is 111 characters long (recommended limit is 79) #1084 FILE: tutorial/ovs-sandbox:406: PATH=$builddir/controller:$builddir/controller-vtep:$builddir/northd:$builddir/ic:$builddir/utilities:$PATH Lines checked: 1124, Warnings: 13, Errors: 0 Please check this out. If you feel there has been an error, please email acon...@redhat.com Thanks, 0-day Robot _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev