Michele Baldessari <[email protected]> writes: > On Wed, Jul 17, 2019 at 06:23:56AM -0500, Terry Wilson wrote: >> Is this just waiting on a couple of line length issues to be fixed? Or do >> those really matter? > > Hi Terry, > > I kind of ignored it because the ovn/utilities/ovndb-servers.ocf file > has already a bunch of lines > 79 chars. I can still fix it up if that > is preferred?
Those shouldn't be 'blocking' anything. It's to bring attention (warning vs. error). Maybe it hasn't been looked at by Numan / aginwala yet. CC'd for more attention. > cheers, > Michele > >> >> On Tue, Jul 9, 2019 at 3:10 AM 0-day Robot <[email protected]> wrote: >> >> > Bleep bloop. Greetings Michele Baldessari, 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 89 characters long (recommended limit is 79) >> > #50 FILE: ovn/utilities/ovndb-servers.ocf:545: >> > ocf_log debug "ovndb_servers: Waiting for promotion $host_name as >> > master to complete" >> > >> > WARNING: Line is 82 characters long (recommended limit is 79) >> > #58 FILE: ovn/utilities/ovndb-servers.ocf:553: >> > ocf_log debug "ovndb_servers: Promotion of $host_name as the master >> > completed" >> > >> > Lines checked: 64, Warnings: 2, Errors: 0 >> > >> > >> > Please check this out. If you feel there has been an error, please email >> > [email protected] >> > >> > Thanks, >> > 0-day Robot >> > _______________________________________________ >> > dev mailing list >> > [email protected] >> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev >> > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
