On Fri, Oct 12, 2012 at 02:58:34PM +0900, FUJITA Tomonori wrote: > On Fri, 12 Oct 2012 14:55:00 +0900 > Isaku Yamahata <[email protected]> wrote: > > > On Fri, Oct 12, 2012 at 02:32:42PM +0900, FUJITA Tomonori wrote: > >> On Fri, 12 Oct 2012 14:27:41 +0900 > >> Isaku Yamahata <[email protected]> wrote: > >> > >> > On Fri, Oct 12, 2012 at 02:15:16PM +0900, Isaku Yamahata wrote: > >> >> On Fri, Oct 12, 2012 at 01:21:39PM +0900, FUJITA Tomonori wrote: > >> >> > On Fri, 12 Oct 2012 12:42:29 +0900 > >> >> > Isaku Yamahata <[email protected]> wrote: > >> >> > > >> >> > > On Fri, Oct 12, 2012 at 06:41:32AM +0900, FUJITA Tomonori wrote: > >> >> > >> On Thu, 11 Oct 2012 17:17:14 +0900 > >> >> > >> Isaku Yamahata <[email protected]> wrote: > >> >> > >> > >> >> > >> > quantum and openvswitch need patch. the patched trees are > >> >> > >> > available > >> >> > >> > for convenience. patches devstack is also avaiable. > >> >> > >> > git://github.com/yamahata/ryu.git ryu-gre-tunnel-oct-11-2012 > >> >> > >> > git://github.com/yamahata/openvswitch.git > >> >> > >> > ryu-gre-tunnel-oct-11-2012 > >> >> > >> > >> >> > >> As I said off-line, asking users to use our own version of OVS > >> >> > >> should > >> >> > >> be avoid. It just causes more problems to us. > >> >> > >> > >> >> > >> I don't like but having OVS python library in Ryu is the only > >> >> > >> option > >> >> > >> for now (until OVS with the changes are norm). For the long term, > >> >> > >> hopefully, we could use of-config instead of OVS json rpc. > >> >> > >> > >> >> > >> Please create a branch to add OVS python library to Ryu at > >> >> > >> github. Don't post it as patches to the mailing list. I'll pull it > >> >> > >> directly. > >> >> > > > >> >> > > I uploaded it at > >> >> > > git://github.com/yamahata/ryu.git contrib-ovs-import > >> >> > > It includes ovs python binding + some module load path adjustment. > >> >> > > >> >> > Why we need to adjust runtime directory and make them configurable? > >> >> > > >> >> > All we need to do is always use Ryu's OVS library. > >> >> > >> >> > >> >> When python-openvswitch is also installed in system, we'd like to use > >> >> Ryu's > >> >> than system installed one. > >> >> It's not configurable. By adjustment I mean that moving/putting Ryu's > >> >> directory (more concretely ryu/contrib) before system directory. > >> > > >> > Ah sorry, I misunderstood. there are two issues. > >> > - module load path > >> > This issue is resolved by modifying sys.path. > >> > > >> > - OVS install path > >> > I think you mean this issue. > >> > OVS runtime directory is compile time option and it is configurable. > >> > python binding may read ovsdb schema file (vswitch.ovsschema) which is > >> > installed on system. > >> > >> Hmm users still need to install ovs? Also, users know where to install > >> the file? > >> > >> Can we put the schema file locally? It is updated per version so we > >> can't? > > > > Regarding to schema file, ovs doesn't need to be installed because Ryu tries > > to retrieve db schema from ovsdb-server. > > So it works fine even without the schema file installed local to Ryu. > > (As long as schema is compatible. incompatible changes to db schema is > > another > > story.) > > > > Probably Ryu GRE tunnel only needs ovs_dbdir, and hard-coding other runtime > > directory wouldn't harm. > > I'm not sure what you mean. Why ovs_dbdir is necesary? ovs doesn't > need to be installed?
Okay, I'll hard-code all of them. And let's see the result. -- yamahata ------------------------------------------------------------------------------ Don't let slow site performance ruin your business. Deploy New Relic APM Deploy New Relic app performance management and know exactly what is happening inside your Ruby, Python, PHP, Java, and .NET app Try New Relic at no cost today and get our sweet Data Nerd shirt too! http://p.sf.net/sfu/newrelic-dev2dev _______________________________________________ Ryu-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ryu-devel
