On Mon, Jun 20, 2016 at 10:16:51AM +0000, Wojciechowicz, RobertX wrote: > Hi, > > > -----Original Message----- > > From: Ben Pfaff [mailto:b...@ovn.org] > > Sent: Wednesday, June 8, 2016 10:41 PM > > To: Wojciechowicz, RobertX <robertx.wojciechow...@intel.com> > > Cc: dev@openvswitch.org > > Subject: Re: [ovs-dev] [PATCH v2] ovsdb: Expose vhost-user socket directory > > in ovsdb > > > > On Thu, Jun 02, 2016 at 11:25:56AM +0100, Robert Wojciechowicz wrote: > > > In order to correctly interoperate with Openstack and ODL, > > > the vhost-user socket directory must be exposed from OVS via OVSDB. > > > Different distros may package OVS in different ways, > > > so the locations of these sockets may vary depending on how > > > ovs-vswitchd has been started. Some clients need information where > > > the sockets are located when instantiating Qemu virtual machines. > > > The full vhost-user socket directory is constructed from current > > > OVS working directory and optionally from specified subdirectory. > > > This patch exposes vhost-user socket directory in Open_vSwitch > > > table in other_config column in two following keys: > > > 1. ovs-run-dir - OVS working directory > > > 2. vhost-sock-dir - subdirectory of ovs-run-dir (might be empty) > > > > > > Signed-off-by: Robert Wojciechowicz <robertx.wojciechow...@intel.com> > > > > > > v1->v2 > > > - moving vswitch-idl.h dependency inside #ifdef block > > > - sock_dir_subcomponent initialization with "" > > > > Same comment as v1: architecturally, ovs-vswitchd only reads > > other-config columns, it never writes to them. Please fix. > > If ovs-vswitchd cannot writes to other-config then the only place > for writing default values to this column I can think of is vswitch > startup script ovs-ctl. > Basically I tested in my environment the below solution > and it seems to solve our issue. > Is it acceptable approach?
It looks like you're trying to use other-config to report something, instead of to configure something. That's not what it's for. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
