[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