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 113 characters long (recommended limit is 79) #41 FILE: ovn/utilities/ovn-ctl:64: ovs-appctl -t $OVN_RUNDIR/ovnnb_db.ctl ovsdb-server/set-active-ovsdb-server `cat $ovnnb_active_conf_file` WARNING: Line is 87 characters long (recommended limit is 79) #42 FILE: ovn/utilities/ovn-ctl:65: ovs-appctl -t $OVN_RUNDIR/ovnnb_db.ctl ovsdb-server/connect-active-ovsdb-server WARNING: Line is 113 characters long (recommended limit is 79) #52 FILE: ovn/utilities/ovn-ctl:78: ovs-appctl -t $OVN_RUNDIR/ovnsb_db.ctl ovsdb-server/set-active-ovsdb-server `cat $ovnsb_active_conf_file` WARNING: Line is 87 characters long (recommended limit is 79) #53 FILE: ovn/utilities/ovn-ctl:79: ovs-appctl -t $OVN_RUNDIR/ovnsb_db.ctl ovsdb-server/connect-active-ovsdb-server WARNING: Line is 86 characters long (recommended limit is 79) #62 FILE: ovn/utilities/ovn-ctl:88: ovs-appctl -t $OVN_RUNDIR/ovnnb_db.ctl ovsdb-server/disconnect-active-ovsdb-server WARNING: Line is 86 characters long (recommended limit is 79) #68 FILE: ovn/utilities/ovn-ctl:93: ovs-appctl -t $OVN_RUNDIR/ovnsb_db.ctl ovsdb-server/disconnect-active-ovsdb-server WARNING: Line is 98 characters long (recommended limit is 79) #77 FILE: ovn/utilities/ovn-ctl:247: ovs-appctl -t $OVN_RUNDIR/ovn${1}_db.ctl ovsdb-server/sync-status | awk '{if(NR==1) print $2}' Lines checked: 127, Warnings: 7, Errors: 0 Please check this out. If you feel there has been an error, please email acon...@bytheb.org Thanks, 0-day Robot _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev