Thank you too, Dumitru! Regards, Vladislav Odintsov
On 11 Jan 2023, at 18:01, Dumitru Ceara <[email protected]<mailto:[email protected]>> wrote: On 1/11/23 12:36, Dumitru Ceara wrote: On 1/10/23 23:21, Odintsov Vladislav wrote: I’ve posted a bunch of series/single patches with the backport. Thanks a lot! I pushed the original patch to branches 22.12 and 22.09. I'll check your backport series next. Please note, tests for branch-21.06 are partially failing. I guess this fail was introduced by next backport/commit: [0] Anyway, other backports to this branch fix the entire tests run (it was completely broken before) and I think it can be accepted and this test (or bug in the code?) can be fixed in a separate patch. Ok, I'll see if I can figure out a way to fix this. It's because https://github.com/ovn-org/ovn/commit/384a7c6237da8f88ab68a9abd0982f92d7d8c2d2 changed the pipeline a bit. I posted a 21.06 specific patch here: https://patchwork.ozlabs.org/project/ovn/patch/[email protected]/ It would be great if you could review it. Ok. Per-branch backport series links: branch-22.06: https://patchwork.ozlabs.org/project/ovn/list/?series=335967 branch-22.03: https://patchwork.ozlabs.org/project/ovn/list/?series=335968 branch-21.12: https://patchwork.ozlabs.org/project/ovn/list/?series=335969 branch-21.09: https://patchwork.ozlabs.org/project/ovn/list/?series=335970 branch-21.06: https://patchwork.ozlabs.org/project/ovn/list/?series=335971 branch-21.03: https://patchwork.ozlabs.org/project/ovn/list/?series=335972 branch-20.12: https://patchwork.ozlabs.org/project/ovn/list/?series=335973 branch-20.09: https://patchwork.ozlabs.org/project/ovn/list/?series=335974 branch-20.06: https://patchwork.ozlabs.org/project/ovn/list/?series=335975 branch-20.03: https://patchwork.ozlabs.org/project/ovn/list/?series=335976 Thanks again for the backport work! I pushed all these. I wonder however if you're still using such old releases (like 20.03). Officially we only actively support releases >= LTS (22.03 [1]). Me neither. We run modern versions of OVN. I’ve just decided it would be good to have a positive regression tests for those branches as well. Regards, Dumitru [1] https://www.ovn.org/en/releases/ 0: https://github.com/ovn-org/ovn/commit/18f719968 Regards, Vladislav Odintsov Regards, Dumitru On 10 Jan 2023, at 22:08, Odintsov Vladislav <[email protected]<mailto:[email protected]><mailto:[email protected]>> wrote: I’ve checked and I’ll try to post backport patches to branches 22.06 and older. Some of them have briken ci, I’ll incorporate ci fixes in appropriate patch sets. So, please make a cherry-pick and push directly to 22.12 and 22.09. I’ll post for 22.06, 22.03, and all 21.xx, maybe more :) Regards, Vladislav Odintsov On 10 Jan 2023, at 14:55, Dumitru Ceara <[email protected]<mailto:[email protected]><mailto:[email protected]><mailto:[email protected]>> wrote: On 1/10/23 10:51, Odintsov Vladislav wrote: Hi Mark, Dumitru, Hi Vladislav, thanks for this change! Would you consider backporting this patch down to branches? It seems very useful. I'm OK with that, it's contained enough. While the change applies cleanly to all stable branches down to 22.03, we need custom modifications for older branches <= 22.06. I can push it directly to 22.09 and 22.12 but if you need it further down would it be possible to prepare a backport patch? Thanks, Dumitru Regards, Vladislav Odintsov On 9 Jan 2023, at 22:48, Mark Michelson <[email protected]<mailto:[email protected]><mailto:[email protected]><mailto:[email protected]> <mailto:[email protected]>> wrote: Thanks Dumitru, Acked-by: Mark Michelson <[email protected]<mailto:[email protected]><mailto:[email protected]><mailto:[email protected]> <mailto:[email protected]>> I applied this change to main. On 12/1/22 11:24, Dumitru Ceara wrote: Signed-off-by: Dumitru Ceara <[email protected]<mailto:[email protected]><mailto:[email protected]><mailto:[email protected]> <mailto:[email protected]>> --- ic/ovn-ic.c | 109 +++++++++++++++++++++++++++++++++++++++++++++++++--- 1 file changed, 104 insertions(+), 5 deletions(-) diff --git a/ic/ovn-ic.c b/ic/ovn-ic.c index e5c193d9dc..45daa337ce 100644 --- a/ic/ovn-ic.c +++ b/ic/ovn-ic.c @@ -1866,13 +1866,112 @@ main(int argc, char *argv[]) struct ovsdb_idl_loop ovnisb_idl_loop = OVSDB_IDL_LOOP_INITIALIZER( ovsdb_idl_create(ovn_ic_sb_db, &icsbrec_idl_class, true, true)); - /* ovn-nb db. XXX: add only needed tables and columns */ + /* ovn-nb db. */ struct ovsdb_idl_loop ovnnb_idl_loop = OVSDB_IDL_LOOP_INITIALIZER( - ovsdb_idl_create(ovnnb_db, &nbrec_idl_class, true, true)); - - /* ovn-sb db. XXX: add only needed tables and columns */ + ovsdb_idl_create(ovnnb_db, &nbrec_idl_class, false, true)); + + ovsdb_idl_add_table(ovnnb_idl_loop.idl, &nbrec_table_nb_global); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, &nbrec_nb_global_col_name); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, &nbrec_nb_global_col_options); + + ovsdb_idl_add_table(ovnnb_idl_loop.idl, + &nbrec_table_logical_router_static_route); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_static_route_col_route_table); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_static_route_col_ip_prefix); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_static_route_col_nexthop); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_static_route_col_external_ids); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_static_route_col_options); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_static_route_col_policy); + + ovsdb_idl_add_table(ovnnb_idl_loop.idl, &nbrec_table_logical_router); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_col_name); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_col_static_routes); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_col_ports); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_col_options); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_col_external_ids); + + ovsdb_idl_add_table(ovnnb_idl_loop.idl, &nbrec_table_logical_router_port); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_port_col_name); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_port_col_networks); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_port_col_external_ids); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_router_port_col_options); + + ovsdb_idl_add_table(ovnnb_idl_loop.idl, &nbrec_table_logical_switch); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_switch_col_name); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_switch_col_ports); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_switch_col_other_config); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_switch_col_external_ids); + + ovsdb_idl_add_table(ovnnb_idl_loop.idl, &nbrec_table_logical_switch_port); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_switch_port_col_name); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_switch_port_col_addresses); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_switch_port_col_options); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_switch_port_col_type); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_switch_port_col_up); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_switch_port_col_addresses); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_switch_port_col_enabled); + ovsdb_idl_add_column(ovnnb_idl_loop.idl, + &nbrec_logical_switch_port_col_external_ids); + + /* ovn-sb db. */ struct ovsdb_idl_loop ovnsb_idl_loop = OVSDB_IDL_LOOP_INITIALIZER( - ovsdb_idl_create(ovnsb_db, &sbrec_idl_class, true, true)); + ovsdb_idl_create(ovnsb_db, &sbrec_idl_class, false, true)); + + ovsdb_idl_add_table(ovnsb_idl_loop.idl, &sbrec_table_chassis); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, &sbrec_chassis_col_encaps); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, &sbrec_chassis_col_name); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, &sbrec_chassis_col_hostname); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, &sbrec_chassis_col_other_config); + + ovsdb_idl_add_table(ovnsb_idl_loop.idl, &sbrec_table_encap); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, &sbrec_encap_col_chassis_name); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, &sbrec_encap_col_type); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, &sbrec_encap_col_ip); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, &sbrec_encap_col_options); + + ovsdb_idl_add_table(ovnsb_idl_loop.idl, &sbrec_table_datapath_binding); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, + &sbrec_datapath_binding_col_external_ids); + + ovsdb_idl_add_table(ovnsb_idl_loop.idl, &sbrec_table_port_binding); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, + &sbrec_port_binding_col_datapath); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, + &sbrec_port_binding_col_mac); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, + &sbrec_port_binding_col_options); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, + &sbrec_port_binding_col_logical_port); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, + &sbrec_port_binding_col_external_ids); + ovsdb_idl_add_column(ovnsb_idl_loop.idl, + &sbrec_port_binding_col_chassis); /* Create IDL indexes */ struct ovsdb_idl_index *nbrec_ls_by_name _______________________________________________ dev mailing list [email protected]<mailto:[email protected]><mailto:[email protected]><mailto:[email protected]> <mailto:[email protected]> https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected]<mailto:[email protected]> https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
