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