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