[ovs-dev] [PATCH v1 1/2 ovn] External IP based NAT: Add Columns and CLI

0-day Robot robot at bytheb.org
Sun Jun 28 22:58:22 UTC 2020


Bleep bloop.  Greetings Ankur Sharma, 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:
ERROR: Author Ankur Sharma <svc.mail.git at nutanix.com> needs to sign off.
WARNING: Unexpected sign-offs from developers who are not authors or co-authors or committers: Ankur Sharma <ankur.sharma at nutanix.com>
WARNING: Line is 83 characters long (recommended limit is 79)
#46 FILE: ovn-nb.ovsschema:404:
                                                         "refTable": "Address_Set",

WARNING: Line is 86 characters long (recommended limit is 79)
#51 FILE: ovn-nb.ovsschema:409:
                                                            "refTable": "Address_Set",

WARNING: Line is 81 characters long (recommended limit is 79)
#52 FILE: ovn-nb.ovsschema:410:
                                                            "refType": "strong"},

ERROR: Improper whitespace around control block
#168 FILE: utilities/ovn-nbctl.c:860:
        NBREC_ADDRESS_SET_FOR_EACH(iter, ctx->idl) {

Lines checked: 271, Warnings: 4, Errors: 2


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