[ovs-dev] [PATCH ovn 1/4] Add support for using OVN specific rundirs
0-day Robot
robot at bytheb.org
Tue Aug 13 17:28:08 UTC 2019
Bleep bloop. Greetings Numan Siddique, 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 99 characters long (recommended limit is 79)
#650 FILE: utilities/ovn-ctl:340:
OVS_RUNDIR=${OVS_RUNDIR} start_ovn_daemon "$OVN_NORTHD_PRIORITY" "$OVN_NORTHD_WRAPPER" "$@"
WARNING: Line is 103 characters long (recommended limit is 79)
#659 FILE: utilities/ovn-ctl:362:
OVS_RUNDIR=${OVS_RUNDIR} start_ovn_daemon "$OVN_CONTROLLER_PRIORITY" "$OVN_CONTROLLER_WRAPPER" "$@"
WARNING: Line is 103 characters long (recommended limit is 79)
#668 FILE: utilities/ovn-ctl:389:
OVS_RUNDIR=${OVS_RUNDIR} start_ovn_daemon "$OVN_CONTROLLER_PRIORITY" "$OVN_CONTROLLER_WRAPPER" "$@"
Lines checked: 1002, Warnings: 3, 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