I’ve posted a bunch of series/single patches with the backport.
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.

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

0: https://github.com/ovn-org/ovn/commit/18f719968

Regards,
Vladislav Odintsov

On 10 Jan 2023, at 22:08, Odintsov Vladislav 
<[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]>> 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]>> wrote:

Thanks Dumitru,

Acked-by: Mark Michelson 
<[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]>>
---
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]>
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

Reply via email to