Bleep bloop.  Greetings Xie Liu, 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 summary should end with a dot.
Subject: controller: CT zone allocation for DGP LSPs enabled ACL
WARNING: Line is 82 characters long (recommended limit is 79)
#56 FILE: controller/binding.c:794:
/* Check if a patch port needs CT zone allocation based on 
enable-router-port-acl.

WARNING: Line is 85 characters long (recommended limit is 79)
#76 FILE: controller/binding.c:2275:
                update_local_lports(pb->logical_port, b_ctx_out, 
LPORT_STATUS_BOUND);

WARNING: Line is 80 characters long (recommended limit is 79)
#87 FILE: controller/binding.c:3178:
        if (sbrec_port_binding_is_updated(pb, SBREC_PORT_BINDING_COL_OPTIONS)) {

WARNING: Line is 85 characters long (recommended limit is 79)
#91 FILE: controller/binding.c:3182:
                update_local_lports(pb->logical_port, b_ctx_out, 
LPORT_STATUS_BOUND);

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