diff mbox series

[ovs-dev,v1] Initialize the right database.

Message ID 1550188864-245758-1-git-send-email-ted.elhourani@nutanix.com
State Superseded
Headers show
Series [ovs-dev,v1] Initialize the right database. | expand

Commit Message

Ted Elhourani Feb. 15, 2019, 12:01 a.m. UTC
Use value of db parameter in order to initialize the correct database.

---
 ovn/utilities/ovn-ctl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Ben Pfaff Feb. 15, 2019, 12:42 a.m. UTC | #1
On Fri, Feb 15, 2019 at 12:01:13AM +0000, Ted Elhourani wrote:
> Use value of db parameter in order to initialize the correct database.

Thanks for the patch.  It looks good to me, but it needs a
Signed-off-by.

Q: What's a Signed-off-by and how do I provide one?

    A: Free and open source software projects usually require a contributor to
    provide some assurance that they're entitled to contribute the code that
    they provide.  Some projects, for example, do this with a Contributor
    License Agreement (CLA) or a copyright assignment that is signed on paper
    or electronically.

    For this purpose, Open vSwitch has adopted something called the Developer's
    Certificate of Origin (DCO), which is also used by the Linux kernel and
    originated there.  Informally stated, agreeing to the DCO is the
    developer's way of attesting that a particular commit that they are
    contributing is one that they are allowed to contribute.  You should visit
    https://developercertificate.org/ to read the full statement of the DCO,
    which is less than 200 words long.

    To certify compliance with the Developer's Certificate of Origin for a
    particular commit, just add the following line to the end of your commit
    message, properly substituting your name and email address:

        Signed-off-by: Firstname Lastname <email@example.org>

    Git has special support for adding a Signed-off-by line to a commit
    message: when you run "git commit", just add the -s option, as in "git
    commit -s".  If you use the "git citool" GUI for commits, you can add a
    Signed-off-by line to the commit message by pressing Control+S.  Other Git
    user interfaces may provide similar support.
0-day Robot Feb. 15, 2019, 12:59 a.m. UTC | #2
Bleep bloop.  Greetings Ted Elhourani, I am a robot and I have tried out your patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
ERROR: Author Ted Elhourani <ted.elhourani@nutanix.com> needs to sign off.
Lines checked: 28, Warnings: 0, Errors: 1


Please check this out.  If you feel there has been an error, please email aconole@bytheb.org

Thanks,
0-day Robot
diff mbox series

Patch

diff --git a/ovn/utilities/ovn-ctl b/ovn/utilities/ovn-ctl
index 1e369b1..f1297a2 100755
--- a/ovn/utilities/ovn-ctl
+++ b/ovn/utilities/ovn-ctl
@@ -222,7 +222,7 @@  $cluster_remote_port
     # Initialize the database if it's running standalone,
     # active-passive, or is the first server in a cluster.
     if test -z "$cluster_remote_addr"; then
-        ovn-nbctl init
+        ovn-${db}ctl init
     fi
 
     if test $mode = cluster; then