[ovs-dev] [PATCH ovn v1] utilities: add ovn-sim

0-day Robot robot at bytheb.org
Thu Oct 10 03:56:32 UTC 2019


Bleep bloop.  Greetings Flavio Fernandes, 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 81 characters long (recommended limit is 79)
#160 FILE: Documentation/ref/ovn-sim.1.rst:77:
Simulating hypervisors and and starting ovn controller (via ovn_attach) on each::

WARNING: Line is 89 characters long (recommended limit is 79)
#170 FILE: Documentation/ref/ovn-sim.1.rst:87:
        ovs-vsctl add-port br-int vif$i -- set Interface vif$i external-ids:iface-id=lp$i

WARNING: Line is 92 characters long (recommended limit is 79)
#189 FILE: Documentation/ref/ovn-sim.1.rst:106:
        ovs-vsctl add-port br-int vif$i -- set Interface vif$i external-ids:iface-id=lp$i) &

WARNING: Line is 99 characters long (recommended limit is 79)
#205 FILE: Documentation/ref/ovn-sim.1.rst:122:
    watch 'for i in `seq $n`; do if test `ovn-nbctl lsp-get-up lp$i` != up; then echo $i; fi; done'

WARNING: Line has trailing whitespace
#777 FILE: utilities/ovn-sim.in:278:
                   --pidfile=ovn-controller.${sandbox}.pid 

WARNING: New doc ovn-sim.1.rst not listed in Documentation/automake.mk
Lines checked: 810, Warnings: 6, 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