diff mbox series

[ovs-dev,v3,5/5] ic-sb schema: add index for routes table & document upgrade path

Message ID 20221215170219.3072151-6-odivlad@gmail.com
State Accepted
Headers show
Series OVN IC multiple same routes fixes | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/github-robot-_ovn-kubernetes fail github build: failed

Commit Message

Vladislav Odintsov Dec. 15, 2022, 5:02 p.m. UTC
Add new uniq index to OVN IC Southbound Route table to prevent same
routes installation.
Also document ovn-ic upgrade details & note new schema change to known
possible duplicate records, which can fail the schema convert.

Signed-off-by: Vladislav Odintsov <odivlad@gmail.com>
---
 Documentation/intro/install/ovn-upgrades.rst | 20 ++++++++++++++++++++
 NEWS                                         |  4 ++++
 ovn-ic-sb.ovsschema                          |  6 ++++--
 3 files changed, 28 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/Documentation/intro/install/ovn-upgrades.rst b/Documentation/intro/install/ovn-upgrades.rst
index 4c131987e..d0865ede8 100644
--- a/Documentation/intro/install/ovn-upgrades.rst
+++ b/Documentation/intro/install/ovn-upgrades.rst
@@ -75,6 +75,20 @@  or if you're using a Linux distribution with systemd::
 
     $ sudo systemctl restart ovn-northd
 
+In case your deployment utilizes OVN Interconnection (OVN IC) functionality,
+it is also needed to restart ovn-ic daemons and separately restart ovn-ic
+databases.
+
+You may perform this restart using the ovn-ctl script::
+
+    $ sudo /usr/share/openvswitch/scripts/ovn-ctl restart_ic
+    $ sudo /usr/share/openvswitch/scripts/ovn-ctl restart_ic_ovsdb
+
+or if you're using a Linux distribution with systemd::
+
+    $ sudo systemctl restart ovn-ic
+    $ sudo systemctl restart ovn-ic-db
+
 Schema Change
 ^^^^^^^^^^^^^
 
@@ -102,6 +116,12 @@  of known impactible schema changes and how to fix when error encountered.
 
     $ ovn-sbctl chassis-del <chassis>
 
+#. Release 22.12: index [transit_switch, availability_zone, route_table,
+   ip_prefix, nexthop] added for OVN Interconnection Southbound DB table Route.
+   If there are duplicated records in this table, users are adviced to upgrade
+   ovn-ic daemons in all availability zones first and after that convert OVS
+   schema (restart ovn-ic database daemon).
+
 
 Upgrading OVN Integration
 -------------------------
diff --git a/NEWS b/NEWS
index d46f04943..9e5aae6ce 100644
--- a/NEWS
+++ b/NEWS
@@ -11,6 +11,10 @@  OVN v22.12.0 - xx xxx xxxx
   - Add support for component templates within logical flows and load
     balancers.
   - Add support for remote port mirroring (Experimental).
+  - Add new OVN IC Route table index. This index ensures no duplicate routes
+    can be advertized.  When upgrading to this version user must ensure that
+    all ovn-ic daemons in all availability zones are upgraded before ovn-ic SB
+    database schema is converted.
 
 OVN v22.09.0 - 16 Sep 2022
 --------------------------
diff --git a/ovn-ic-sb.ovsschema b/ovn-ic-sb.ovsschema
index 72c9d3f3e..1d60b36d1 100644
--- a/ovn-ic-sb.ovsschema
+++ b/ovn-ic-sb.ovsschema
@@ -1,7 +1,7 @@ 
 {
     "name": "OVN_IC_Southbound",
-    "version": "1.1.0",
-    "cksum": "2309827842 6784",
+    "version": "1.1.1",
+    "cksum": "3684563024 6914",
     "tables": {
         "IC_SB_Global": {
             "columns": {
@@ -101,6 +101,8 @@ 
                 "external_ids": {
                     "type": {"key": "string", "value": "string",
                              "min": 0, "max": "unlimited"}}},
+            "indexes": [["transit_switch", "availability_zone", "route_table",
+                         "ip_prefix", "nexthop"]],
             "isRoot": true},
         "Connection": {
             "columns": {