Re: [ovs-dev] [PATCH] vtep: Introduce other_config column for the Global table.

2016-05-11 Thread Dennis Sam
Oh sorry, there was some co-ordination in terms of content but not it in
terms of who was going to do it. :)

On Wed, May 11, 2016 at 3:32 PM, Justin Pettit  wrote:

>
> > On May 11, 2016, at 3:30 PM, Saurabh Shrivastava <
> saurabh.shrivast...@nuagenetworks.net> wrote:
> >
> > Lets go with Dennis's patch.
>
> Sounds good.  They're identical down to the checksum, so it makes it easy
> to see there's no difference.  :-)
>
> Thanks,
>
> --Justin
>
>
>
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev


Re: [ovs-dev] [PATCH] vtep: Introduce other_config column for the Global table.

2016-05-11 Thread Justin Pettit

> On May 11, 2016, at 3:30 PM, Saurabh Shrivastava 
>  wrote:
> 
> Lets go with Dennis's patch. 

Sounds good.  They're identical down to the checksum, so it makes it easy to 
see there's no difference.  :-)

Thanks,

--Justin


___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev


Re: [ovs-dev] [PATCH] vtep: Introduce other_config column for the Global table.

2016-05-11 Thread Saurabh Shrivastava
Lets go with Dennis's patch.

On Wed, May 11, 2016 at 3:19 PM, Justin Pettit  wrote:

> This seems to be an identical patch to one that Dennis Sam (cc'd)
> submitted a few hours ago.  Did the two of you coordinate or is this just
> coincidence?  I'm just curious how we want to handle authorship.
>
> I'm cc'ing Bruce and Anupam to get their take on the patch.
>
> --Justin
>
>
> > On May 11, 2016, at 3:15 PM, Saurabh Shrivastava <
> saurabh.shrivast...@nuagenetworks.net> wrote:
> >
> > Signed-off-by: Saurabh Shrivastava <
> saurabh.shrivast...@nuagenetworks.net>
> > ---
> > vtep/vtep.ovsschema | 7 +--
> > vtep/vtep.xml   | 8 
> > 2 files changed, 13 insertions(+), 2 deletions(-)
> >
> > diff --git a/vtep/vtep.ovsschema b/vtep/vtep.ovsschema
> > index 533fd2e..799d760 100644
> > --- a/vtep/vtep.ovsschema
> > +++ b/vtep/vtep.ovsschema
> > @@ -1,6 +1,6 @@
> > {
> >   "name": "hardware_vtep",
> > -  "cksum": "770244945 3",
> > +  "cksum": "841956498 11245",
> >   "tables": {
> > "Global": {
> >   "columns": {
> > @@ -10,6 +10,9 @@
> >"min": 0, "max": "unlimited"}},
> > "switches": {
> >   "type": {"key": {"type": "uuid", "refTable":
> "Physical_Switch"},
> > +   "min": 0, "max": "unlimited"}},
> > +"other_config": {
> > +  "type": {"key": "string", "value": "string",
> >"min": 0, "max": "unlimited"}}
> >   },
> >   "maxRows": 1,
> > @@ -296,4 +299,4 @@
> >   "ephemeral": true}},
> >   "indexes": [["target"]],
> >   "isRoot": false}},
> > -  "version": "1.5.1"}
> > +  "version": "1.5.2"}
> > diff --git a/vtep/vtep.xml b/vtep/vtep.xml
> > index a3a6988..39d9bd3 100644
> > --- a/vtep/vtep.xml
> > +++ b/vtep/vtep.xml
> > @@ -88,6 +88,14 @@
> > table for more information.
> >   
> > 
> > +
> > +
> > +  The overall purpose of this column is described under Common
> > +  Column at the beginning of this document.
> > +
> > +  
> > +
> > +
> >   
> >
> >   
> > --
> > 1.8.3.1
> > ___
> > dev mailing list
> > dev@openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
>
>


-- 
Saurabh
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev


Re: [ovs-dev] [PATCH] vtep: Introduce other_config column for the Global table.

2016-05-11 Thread Justin Pettit
This seems to be an identical patch to one that Dennis Sam (cc'd) submitted a 
few hours ago.  Did the two of you coordinate or is this just coincidence?  I'm 
just curious how we want to handle authorship.

I'm cc'ing Bruce and Anupam to get their take on the patch.

--Justin


> On May 11, 2016, at 3:15 PM, Saurabh Shrivastava 
>  wrote:
> 
> Signed-off-by: Saurabh Shrivastava 
> ---
> vtep/vtep.ovsschema | 7 +--
> vtep/vtep.xml   | 8 
> 2 files changed, 13 insertions(+), 2 deletions(-)
> 
> diff --git a/vtep/vtep.ovsschema b/vtep/vtep.ovsschema
> index 533fd2e..799d760 100644
> --- a/vtep/vtep.ovsschema
> +++ b/vtep/vtep.ovsschema
> @@ -1,6 +1,6 @@
> {
>   "name": "hardware_vtep",
> -  "cksum": "770244945 3",
> +  "cksum": "841956498 11245",
>   "tables": {
> "Global": {
>   "columns": {
> @@ -10,6 +10,9 @@
>"min": 0, "max": "unlimited"}},
> "switches": {
>   "type": {"key": {"type": "uuid", "refTable": "Physical_Switch"},
> +   "min": 0, "max": "unlimited"}},
> +"other_config": {
> +  "type": {"key": "string", "value": "string",
>"min": 0, "max": "unlimited"}}
>   },
>   "maxRows": 1,
> @@ -296,4 +299,4 @@
>   "ephemeral": true}},
>   "indexes": [["target"]],
>   "isRoot": false}},
> -  "version": "1.5.1"}
> +  "version": "1.5.2"}
> diff --git a/vtep/vtep.xml b/vtep/vtep.xml
> index a3a6988..39d9bd3 100644
> --- a/vtep/vtep.xml
> +++ b/vtep/vtep.xml
> @@ -88,6 +88,14 @@
> table for more information.
>   
> 
> +
> +
> +  The overall purpose of this column is described under Common
> +  Column at the beginning of this document.
> +
> +  
> +
> +
>   
> 
>   
> -- 
> 1.8.3.1
> ___
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev

___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev


[ovs-dev] [PATCH] vtep: Introduce other_config column for the Global table.

2016-05-11 Thread Saurabh Shrivastava
Signed-off-by: Saurabh Shrivastava 
---
 vtep/vtep.ovsschema | 7 +--
 vtep/vtep.xml   | 8 
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/vtep/vtep.ovsschema b/vtep/vtep.ovsschema
index 533fd2e..799d760 100644
--- a/vtep/vtep.ovsschema
+++ b/vtep/vtep.ovsschema
@@ -1,6 +1,6 @@
 {
   "name": "hardware_vtep",
-  "cksum": "770244945 3",
+  "cksum": "841956498 11245",
   "tables": {
 "Global": {
   "columns": {
@@ -10,6 +10,9 @@
"min": 0, "max": "unlimited"}},
 "switches": {
   "type": {"key": {"type": "uuid", "refTable": "Physical_Switch"},
+   "min": 0, "max": "unlimited"}},
+"other_config": {
+  "type": {"key": "string", "value": "string",
"min": 0, "max": "unlimited"}}
   },
   "maxRows": 1,
@@ -296,4 +299,4 @@
   "ephemeral": true}},
   "indexes": [["target"]],
   "isRoot": false}},
-  "version": "1.5.1"}
+  "version": "1.5.2"}
diff --git a/vtep/vtep.xml b/vtep/vtep.xml
index a3a6988..39d9bd3 100644
--- a/vtep/vtep.xml
+++ b/vtep/vtep.xml
@@ -88,6 +88,14 @@
 table for more information.
   
 
+
+
+  The overall purpose of this column is described under Common
+  Column at the beginning of this document.
+
+  
+
+
   

   
-- 
1.8.3.1
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev