Re: [ovs-dev] [PATCH ovn] Enable OVN in tutorial/ovs-sandbox by default

2019-08-07 Thread Numan Siddique
On Wed, Aug 7, 2019 at 11:27 PM Mark Michelson  wrote:

> Acked-by: Mark Michelson 
>
>
Thanks for the review. I applied this to master.

Numan


> On 8/7/19 11:06 AM, nusid...@redhat.com wrote:
> > From: Numan Siddique 
> >
> > The patch removs the --ovn option and enables OVN by default.
> >
> > Signed-off-by: Numan Siddique 
> > ---
> >   tutorial/ovs-sandbox | 6 +-
> >   1 file changed, 1 insertion(+), 5 deletions(-)
> >
> > diff --git a/tutorial/ovs-sandbox b/tutorial/ovs-sandbox
> > index 9b4c3e4f5..47032b499 100755
> > --- a/tutorial/ovs-sandbox
> > +++ b/tutorial/ovs-sandbox
> > @@ -67,7 +67,7 @@ srcdir=
> >   schema=
> >   installed=false
> >   built=false
> > -ovn=false
> > +ovn=true
> >   ovnsb_schema=
> >   ovnnb_schema=
> >   ovn_rbac=true
> > @@ -129,7 +129,6 @@ General options:
> > -S, --schema=FILEuse FILE as vswitch.ovsschema
> >
> >   OVN options:
> > -  -o, --ovnenable OVN
> > --no-ovn-rbacdisable role-based access control for OVN
> > --n-northds=NUMBER   run NUMBER copies of northd (default: 1)
> > --nbdb-model=standalone|backup|clusterednorthbound database model
> > @@ -201,9 +200,6 @@ EOF
> >   --gdb-ovn-controller-vtep)
> >   gdb_ovn_controller_vtep=true
> >   ;;
> > --o|--ovn)
> > -ovn=true
> > -;;
> >   --no-ovn-rbac)
> >   ovn_rbac=false
> >   ;;
> >
>
>
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH ovn] Enable OVN in tutorial/ovs-sandbox by default

2019-08-07 Thread Mark Michelson

Acked-by: Mark Michelson 

On 8/7/19 11:06 AM, nusid...@redhat.com wrote:

From: Numan Siddique 

The patch removs the --ovn option and enables OVN by default.

Signed-off-by: Numan Siddique 
---
  tutorial/ovs-sandbox | 6 +-
  1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/tutorial/ovs-sandbox b/tutorial/ovs-sandbox
index 9b4c3e4f5..47032b499 100755
--- a/tutorial/ovs-sandbox
+++ b/tutorial/ovs-sandbox
@@ -67,7 +67,7 @@ srcdir=
  schema=
  installed=false
  built=false
-ovn=false
+ovn=true
  ovnsb_schema=
  ovnnb_schema=
  ovn_rbac=true
@@ -129,7 +129,6 @@ General options:
-S, --schema=FILEuse FILE as vswitch.ovsschema
  
  OVN options:

-  -o, --ovnenable OVN
--no-ovn-rbacdisable role-based access control for OVN
--n-northds=NUMBER   run NUMBER copies of northd (default: 1)
--nbdb-model=standalone|backup|clusterednorthbound database model
@@ -201,9 +200,6 @@ EOF
  --gdb-ovn-controller-vtep)
  gdb_ovn_controller_vtep=true
  ;;
--o|--ovn)
-ovn=true
-;;
  --no-ovn-rbac)
  ovn_rbac=false
  ;;



___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


[ovs-dev] [PATCH ovn] Enable OVN in tutorial/ovs-sandbox by default

2019-08-07 Thread nusiddiq
From: Numan Siddique 

The patch removs the --ovn option and enables OVN by default.

Signed-off-by: Numan Siddique 
---
 tutorial/ovs-sandbox | 6 +-
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/tutorial/ovs-sandbox b/tutorial/ovs-sandbox
index 9b4c3e4f5..47032b499 100755
--- a/tutorial/ovs-sandbox
+++ b/tutorial/ovs-sandbox
@@ -67,7 +67,7 @@ srcdir=
 schema=
 installed=false
 built=false
-ovn=false
+ovn=true
 ovnsb_schema=
 ovnnb_schema=
 ovn_rbac=true
@@ -129,7 +129,6 @@ General options:
   -S, --schema=FILEuse FILE as vswitch.ovsschema
 
 OVN options:
-  -o, --ovnenable OVN
   --no-ovn-rbacdisable role-based access control for OVN
   --n-northds=NUMBER   run NUMBER copies of northd (default: 1)
   --nbdb-model=standalone|backup|clusterednorthbound database model
@@ -201,9 +200,6 @@ EOF
 --gdb-ovn-controller-vtep)
 gdb_ovn_controller_vtep=true
 ;;
--o|--ovn)
-ovn=true
-;;
 --no-ovn-rbac)
 ovn_rbac=false
 ;;
-- 
2.21.0

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev