[ovs-dev] [PATCH ovn] ovn-ctl: introduce ovsdb-{n, s}b-wrapper options

0-day Robot robot at bytheb.org
Tue Jul 21 14:57:22 UTC 2020


Bleep bloop.  Greetings Lorenzo Bianconi, 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)
#47 FILE: utilities/ovn-ctl:299:
                log_failure_msg "valgrind not installed, running $daemon without it"

WARNING: Line is 82 characters long (recommended limit is 79)
#59 FILE: utilities/ovn-ctl:311:
                log_failure_msg "strace not installed, running $daemon without it"

WARNING: Line is 82 characters long (recommended limit is 79)
#65 FILE: utilities/ovn-ctl:317:
            log_failure_msg "unknown wrapper $wrapper, running $daemon without it"

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