[ovs-dev] [PATCH 4/4] rhel: run ovn with the same user as ovs

0-day Robot robot at bytheb.org
Wed May 8 13:14:52 UTC 2019


Bleep bloop.  Greetings Jaime Caamaño Ruiz, 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 129 characters long (recommended limit is 79)
#93 FILE: rhel/usr_lib_systemd_system_ovsdb-server.service:15:
ExecStartPre=/bin/sh -c 'rm -f /run/openvswitch.useropts; /usr/bin/echo "OVS_USER_ID=${OVS_USER_ID}" > /run/openvswitch.useropts'

WARNING: Line is 158 characters long (recommended limit is 79)
#94 FILE: rhel/usr_lib_systemd_system_ovsdb-server.service:16:
ExecStartPre=/bin/sh -c 'if [ "$${OVS_USER_ID/:*/}" != "root" ]; then /usr/bin/echo "OVS_USER_OPT=--ovs-user=${OVS_USER_ID}" >> /run/openvswitch.useropts; fi'

Lines checked: 110, Warnings: 2, Errors: 0


Please check this out.  If you feel there has been an error, please email aconole at bytheb.org

Thanks,
0-day Robot


More information about the dev mailing list