Re: [ovs-dev] [PATCH] vtep: Introduce other_config column for the Global table.
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.
> 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.
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.
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.
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