diff mbox

[ovs-dev,1/4] Update OVN SB to use physical endpoints table

Message ID 1456857559-99437-2-git-send-email-dball@vmware.com
State Changes Requested
Headers show

Commit Message

Darrell Ball March 1, 2016, 6:39 p.m. UTC 
Signed-off-by: Darrell Ball <dball@vmware.com>
---
 ovn/ovn-sb.ovsschema | 21 ++++++++++++++++++++-
 1 file changed, 20 insertions(+), 1 deletion(-)

Comments

Ben Pfaff March 18, 2016, 6:06 p.m. UTC | #1 
On Tue, Mar 01, 2016 at 10:39:16AM -0800, Darrell Ball wrote:
> Signed-off-by: Darrell Ball <dball@vmware.com>

This patch isn't self-contained because it causes a build failure:

      GEN      ovn/ovn-sb.5
    ../ovsdb/ovsdb-doc: undocumented table Physical_Endpoint

It should be squashed in with the next patch, I guess.  I'll provide
more comments as I review that patch.
Darrell Ball March 18, 2016, 9:19 p.m. UTC | #2 
On 3/18/16, 11:06 AM, "dev on behalf of Ben Pfaff" <dev-bounces@openvswitch.org on behalf of blp@ovn.org> wrote:

>On Tue, Mar 01, 2016 at 10:39:16AM -0800, Darrell Ball wrote:

>> Signed-off-by: Darrell Ball <dball@vmware.com>

>

>This patch isn't self-contained because it causes a build failure:

>

>      GEN      ovn/ovn-sb.5

>    ../ovsdb/ovsdb-doc: undocumented table Physical_Endpoint

>

>It should be squashed in with the next patch, I guess.  I'll provide

>more comments as I review that patch.


Thanks

I’m preparing a single non-RFC patch for physical-logical separation, localnet and L2 gateway
so that all dependencies are met


>_______________________________________________

>dev mailing list

>dev@openvswitch.org

>http://openvswitch.org/mailman/listinfo/dev
diff mbox

Patch

diff --git a/ovn/ovn-sb.ovsschema b/ovn/ovn-sb.ovsschema
index a9a91e5..dbbc7ab 100644
--- a/ovn/ovn-sb.ovsschema
+++ b/ovn/ovn-sb.ovsschema
@@ -1,7 +1,7 @@ 
 {
     "name": "OVN_Southbound",
     "version": "1.0.0",
-    "cksum": "1392129391 5060",
+    "cksum": "3279910014 6104",
     "tables": {
         "Chassis": {
             "columns": {
@@ -70,6 +70,21 @@ 
                              "min": 0, "max": "unlimited"}}},
             "indexes": [["tunnel_key"]],
             "isRoot": true},
+        "Physical_Endpoint": {
+            "columns": {
+                "name": {"type": "string"},
+                "chassis": {"type": {"key": {"type": "uuid",
+                                             "refTable": "Chassis",
+                                             "refType": "weak"},
+                                     "min": 0, "max": 1}},
+                "chassis_port": {"type": {"key": "string", "min": 0, "max": 1}},
+                "type": {"type": {"key": {
+                           "type": "string",
+                           "enum": ["set", ["vlan", "mpls_label_stack", "later"]]}}},
+                "ingress_encap": {"type": "string"},
+                "egress_encap": {"type": "string"}},
+            "indexes": [["name"]],
+            "isRoot": true},
         "Port_Binding": {
             "columns": {
                 "logical_port": {"type": "string"},
@@ -95,6 +110,10 @@ 
                                              "refTable": "Chassis",
                                              "refType": "weak"},
                                      "min": 0, "max": 1}},
+                "phys_endpt": {"type": {"key": {"type": "uuid",
+                                             "refTable": "Physical_Endpoint",
+                                             "refType": "weak"},
+                                     "min": 0, "max": 1}},
                 "mac": {"type": {"key": "string",
                                  "min": 0,
                                  "max": "unlimited"}}},