From: Han Zhou <hzh...@ebay.com>
When adding a new chassis, if there is an old chassis with same IP
existed in Encap table, it is allowed to be added today. However,
allowing it to be added results in problems:
1. The new chassis cannot work because none of the other chassises
are able to create tunnel to it, because of the IP confliction
with already existed tunnel to the old chassis.
2. All the other chassises will continuously retry creating the tunnel
and complaining about the error.
So, instead of hiding the problem, it is better to expose it while
trying to add the second chassis with duplicated IP. This patch
ensures it from the ovsdb schema.
Signed-off-by: Han Zhou <hzh...@ebay.com>
---
v2 -> v3: update NEWS for the potential DB upgrading failure
NEWS | 5 +++++
ovn/ovn-sb.ovsschema | 7 ++++---
2 files changed, 9 insertions(+), 3 deletions(-)
diff --git a/NEWS b/NEWS
index 02402d1..b53978d 100644
--- a/NEWS
+++ b/NEWS
@@ -7,6 +7,11 @@ Post-v2.10.0
- The environment variable OVS_CTL_TIMEOUT, if set, is now used
as the default timeout for control utilities.
- ovn:
+ * OVN-SB schema changed: duplicated IP with same Encapsulation type
+ is not allowed any more. DB upgrading from an earlier version
+ could fail if there were already duplicated entries. In such case,
+ user can delete the entries using command "ovn-sbctl chassis-del
<chassis>"
+ to delete them before upgrading.
* New support for IPSEC encrypted tunnels between hypervisors.
* ovn-ctl: allow passing user:group ids to the OVN daemons.
- DPDK:
diff --git a/ovn/ovn-sb.ovsschema b/ovn/ovn-sb.ovsschema
index 5b9537f..afa9859 100644
--- a/ovn/ovn-sb.ovsschema
+++ b/ovn/ovn-sb.ovsschema
@@ -1,7 +1,7 @@
{
"name": "OVN_Southbound",
- "version": "1.17.0",
- "cksum": "3217981733 15045",
+ "version": "1.18.0",
+ "cksum": "910560265 15086",
"tables": {
"SB_Global": {
"columns": {
@@ -50,7 +50,8 @@
"min": 0,
"max": "unlimited"}},
"ip": {"type": "string"},
- "chassis_name": {"type": "string"}}},
+ "chassis_name": {"type": "string"}},
+ "indexes": [["type", "ip"]]},
"Address_Set": {
"columns": {
"name": {"type": "string"},
--
2.1.0
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
