[ovs-dev] [PATCH ovn v2 2/3] rhel: add systemd-unit for ovn-ic and move IC to sub-rpm ovn-ic

0-day Robot robot at bytheb.org
Tue Dec 22 09:01:34 UTC 2020


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 143 characters long (recommended limit is 79)
#174 FILE: rhel/usr_lib_systemd_system_ovn-ic.service:10:
#   Environment="OVN_IC_OPTS=--db-ic-nb-sock=/usr/local/var/run/ovn/ovn_ic_nb_db.sock --db-ic-sb-sock=/usr/local/var/run/ovn/ovn_ic_sb_db.sock"

WARNING: Line is 89 characters long (recommended limit is 79)
#176 FILE: rhel/usr_lib_systemd_system_ovn-ic.service:12:
# Alternatively, you may specify environment variables in the file /etc/sysconfig/ovn-ic:

WARNING: Line is 131 characters long (recommended limit is 79)
#178 FILE: rhel/usr_lib_systemd_system_ovn-ic.service:14:
#   OVN_IC_OPTS="--db-ic-nb-sock=/usr/local/var/run/ovn/ovn_ic_nb_db.sock --db-ic-sb-sock=/usr/local/var/run/ovn/ovn_ic_sb_db.sock"

Lines checked: 199, 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