[ovs-dev] [PATCH 3/3] ovn-ctl: add support for SSL nb/sb db connections

Lance Richardson lrichard at redhat.com
Thu Dec 8 15:04:09 UTC 2016


> From: "Russell Bryant" <russell at ovn.org>

> What would you think of changing the DEFAULT_REMOTE option to be
> CREATE_REMOTE to more clearly indicate that the option is telling ovn-ctl
> that we want it to go ahead and set up a remote for us.
> 

Sounds good, I think CREATE_REMOTE is a better name.

> When CREATE_REMOTE is "yes", instead of adding it to the ovsdb-server
> command line, how about we add it to to the Connection table.  That way we
> can add support for some additional options.  This came up from our need to

Makes sense, having the remote configuration in the db would give us more
flexibility.

> set the inactivity_probe option.  Presumably we could add some more options
> to this script to let you enable SSL through options, as well.
> 
> Numan had a start at a patch doing something like this here, but it
> probably makes sense to just integrate with your patch.
> 
> https://github.com/numansiddique/overcloud_image_for_ovn/blob/master/patches/ovs-0005-ovn-ctl-Add-remotes-in-Connection-table-of-NB-and-SB.patch
> 

Thanks, I will steal some of that :-)

I'll incorporate and post v2 shortly.

Thanks,

   Lance

> 
> > Regards,
> >
> >    Lance
> >
> > > Some thing like below
> > >
> > > #########
> > > if test X"$DB_NB_DEFAULT_REMOTE" = Xno; then
> > >    ovn-nbctl set-connection "ptcp:$DB_NB_PORT:$DB_NB_ADDR"
> > > fi
> > > ​#########
> > >
> > > ​Thanks
> > > Numan
> > > ​
> > >
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> >
> 
> 
> 
> --
> Russell Bryant
> 


More information about the dev mailing list