Bleep bloop.  Greetings Ben Pfaff, 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 80 characters long (recommended limit is 79)
#87 FILE: utilities/ovn-nbctl.c:823:
    ovsdb_idl_add_column(ctx->idl, &nbrec_logical_switch_port_col_external_ids);

WARNING: Line is 83 characters long (recommended limit is 79)
#102 FILE: utilities/ovn-nbctl.c:838:
    ovsdb_idl_add_column(ctx->idl, 
&nbrec_logical_router_port_col_gateway_chassis);

WARNING: Line is 90 characters long (recommended limit is 79)
#1308 FILE: utilities/ovn-nbctl.c:6531:
    { "ls-del", 1, 1, "SWITCH", nbctl_pre_ls_del, nbctl_ls_del, NULL, 
"--if-exists", RW },

WARNING: Line is 80 characters long (recommended limit is 79)
#1563 FILE: utilities/ovn-nbctl.c:6688:
    { "lb-list", 0, 1, "[LB]", nbctl_pre_lb_list, nbctl_lb_list, NULL, "", RO },

WARNING: Line is 80 characters long (recommended limit is 79)
#1634 FILE: utilities/ovn-nbctl.c:6745:
     pre_ha_ch_grp_remove_chassis, cmd_ha_ch_grp_remove_chassis, NULL, "", RW },

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