Bleep bloop.  Greetings Dmitrii S., 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: The subject, '<area>: <summary>', is over 70 characters, i.e., 72.
Subject: controller: Skip LB route install when local Service_Monitor is 
offline.
WARNING: Line is 82 characters long (recommended limit is 79)
#165 FILE: controller/route.c:404:
                /* If the route carries the full backend service selector, look 
up

WARNING: Line is 81 characters long (recommended limit is 79)
#166 FILE: controller/route.c:405:
                 * matching Service_Monitor rows on this chassis. Skip 
installing

WARNING: Line is 82 characters long (recommended limit is 79)
#168 FILE: controller/route.c:407:
                 * Routes without the full selector are installed 
unconditionally.

WARNING: Line is 83 characters long (recommended limit is 79)
#170 FILE: controller/route.c:409:
                 * The full selector (ip + port + protocol) is required to 
identify

WARNING: Line is 81 characters long (recommended limit is 79)
#171 FILE: controller/route.c:410:
                 * the row as LB-derived, since route_source is not stored on 
the

WARNING: Line is 81 characters long (recommended limit is 79)
#172 FILE: controller/route.c:411:
                 * SB row.  A partial selector could match an unrelated SM on 
the

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

Reply via email to