diff mbox series

[ovs-dev] tutorial: skip passing .db for backup option for ovn_start_ovsdb_server:

Message ID 1523493781-12112-1-git-send-email-aginwala@ebay.com
State Accepted
Headers show
Series [ovs-dev] tutorial: skip passing .db for backup option for ovn_start_ovsdb_server: | expand

Commit Message

aginwala aginwala April 12, 2018, 12:43 a.m. UTC
current params uses two sb1.db which is redundant:
e.g. ovsdb-server --remote=punix:sb1.ovsdb sb1.db sb1.db
expected:
e.g. ovsdb-server --remote=punix:sb1.ovsdb sb1.db

tested and works as expected:
ovn-sbctl --db=unix:/root/ovs/tutorial/sandbox/sb2.ovsdb show
Chassis "chassis-1"
    hostname: sandbox
    Encap geneve
        ip: "127.0.0.1"
        options: {csum="true"}

Signed-off-by: aginwala <aginwala@ebay.com>
---
 tutorial/ovs-sandbox | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Ben Pfaff April 18, 2018, 11:23 p.m. UTC | #1
On Wed, Apr 11, 2018 at 05:43:01PM -0700, aginwala wrote:
> current params uses two sb1.db which is redundant:
> e.g. ovsdb-server --remote=punix:sb1.ovsdb sb1.db sb1.db
> expected:
> e.g. ovsdb-server --remote=punix:sb1.ovsdb sb1.db
> 
> tested and works as expected:
> ovn-sbctl --db=unix:/root/ovs/tutorial/sandbox/sb2.ovsdb show
> Chassis "chassis-1"
>     hostname: sandbox
>     Encap geneve
>         ip: "127.0.0.1"
>         options: {csum="true"}
> 
> Signed-off-by: aginwala <aginwala@ebay.com>

Thanks, applied to master.
diff mbox series

Patch

diff --git a/tutorial/ovs-sandbox b/tutorial/ovs-sandbox
index 7a9c195..feba578 100755
--- a/tutorial/ovs-sandbox
+++ b/tutorial/ovs-sandbox
@@ -436,7 +436,7 @@  if $ovn; then
                 for i in 1 2; do
                     run ovsdb-tool create $db$i.db "$schema"
                 done
-                ovn_start_ovsdb_server 1 ${db}1.db
+                ovn_start_ovsdb_server 1
                 ovn_start_ovsdb_server 2 --sync-from=unix:${db}1.ovsdb
                 remote=unix:${db}1.ovsdb
                 backup_note="$backup_note