Bleep bloop. Greetings Mark Michelson, 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 84 characters long (recommended limit is 79)
#368 FILE: controller/pinctrl.c:5672:
hmap_insert(chassis_mac_bindings, &ch_binding->hmap_node,
ch_binding->hash);
WARNING: Line is 82 characters long (recommended limit is 79)
#378 FILE: controller/pinctrl.c:5682:
struct ovsdb_idl_index
*sbrec_mac_binding_by_lport_ip,
WARNING: Line is 83 characters long (recommended limit is 79)
#416 FILE: controller/pinctrl.c:5720:
struct ovsdb_idl_index
*sbrec_mac_binding_by_lport_ip,
WARNING: Line is 86 characters long (recommended limit is 79)
#495 FILE: controller/pinctrl.c:5810:
install_chassis_mac_bindings(ovnsb_idl_txn,
sbrec_mac_binding_by_lport_ip,
WARNING: Line is 86 characters long (recommended limit is 79)
#509 FILE: controller/pinctrl.c:5824:
install_chassis_mac_bindings(ovnsb_idl_txn,
sbrec_mac_binding_by_lport_ip,
WARNING: Line is 80 characters long (recommended limit is 79)
#603 FILE: ovn-sb.ovsschema:244:
"refTable": "Datapath_Binding"}}},
Lines checked: 838, Warnings: 6, Errors: 0
build:
-e 's,[@]OVN_RUNDIR[@],/usr/local/var/run/ovn,g' \
-e
's,[@]OVSBUILDDIR[@],/var/lib/jenkins/jobs/0day_robot_upstream_build_ovn_from_pw/workspace/OVSDIR,g'
\
-e 's,[@]VERSION[@],21.03.90,g' \
-e 's,[@]OVSVERSION[@],2.15.90,g' \
-e 's,[@]localstatedir[@],/usr/local/var,g' \
-e 's,[@]pkgdatadir[@],/usr/local/share/ovn,g' \
-e 's,[@]sysconfdir[@],/usr/local/etc,g' \
-e 's,[@]bindir[@],/usr/local/bin,g' \
-e 's,[@]sbindir[@],/usr/local/sbin,g' \
-e
's,[@]abs_builddir[@],/var/lib/jenkins/jobs/0day_robot_upstream_build_ovn_from_pw/workspace,g'
\
-e
's,[@]abs_top_srcdir[@],/var/lib/jenkins/jobs/0day_robot_upstream_build_ovn_from_pw/workspace,g'
\
> utilities/ovn-lib.tmp
mv utilities/ovn-lib.tmp utilities/ovn-lib
/bin/sh
/var/lib/jenkins/jobs/0day_robot_upstream_build_ovn_from_pw/workspace/build-aux/missing
autom4te --language=autotest -I '.' -o tests/testsuite.tmp tests/testsuite.at
mv tests/testsuite.tmp tests/testsuite
/bin/sh
/var/lib/jenkins/jobs/0day_robot_upstream_build_ovn_from_pw/workspace/build-aux/missing
autom4te --language=autotest -I '.' -o tests/system-kmod-testsuite.tmp
tests/system-kmod-testsuite.at
mv tests/system-kmod-testsuite.tmp tests/system-kmod-testsuite
/bin/sh
/var/lib/jenkins/jobs/0day_robot_upstream_build_ovn_from_pw/workspace/build-aux/missing
autom4te --language=autotest -I '.' -o tests/system-userspace-testsuite.tmp
tests/system-userspace-testsuite.at
mv tests/system-userspace-testsuite.tmp tests/system-userspace-testsuite
tests/ofproto-macros.at
See above for files that use tabs for indentation.
Please use spaces instead.
make[1]: *** [check-tabs] Error 1
make[1]: Leaving directory
`/var/lib/jenkins/jobs/0day_robot_upstream_build_ovn_from_pw/workspace'
make: *** [all] Error 2
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
