On Tue, Oct 15, 2019 at 10:38 AM Ben Pfaff <b...@ovn.org> wrote:
> On Mon, Oct 14, 2019 at 07:45:11PM -0700, aginwala wrote: > > On Mon, Oct 14, 2019 at 4:38 PM Ben Pfaff <b...@ovn.org> wrote: > > > > > On Mon, Oct 14, 2019 at 03:33:25PM -0700, Ben Pfaff wrote: > > > > On Wed, Oct 09, 2019 at 04:56:40PM -0700, amgin...@gmail.com wrote: > > > > > From: Aliasgar Ginwala <aginw...@ebay.com> > > > > > > > > > > Add new env variables OVN_NBCTL_OPTIONS and OVN_SBCTL_OPTIONS for > > > > > ovn-nbctl and ovn-sbctl respectively where user can set any single > > > > > supported option. e.g export OVN_NBCTL_OPTIONS=--no-leader-only. > > > > > Above env var OVN_NBCTL_OPTIONS have no effect if user runs > > > > > command as ovn-nbctl --no-leader-only <command> > > > > > > > > > > Signed-off-by: Aliasgar Ginwala <aginw...@ebay.com> > > > > > > > > I think that this could be factored out into lib/command-line.c > rather > > > > than done as a copy-and-paste into two files. > > > > > > > > I don't think that the ops_passed variable needs to be static. > > > > > > Also, I'd consider giving an example in the documentation of how to use > > > this new variable. The purpose is unclear enough that I might not be > > > able to quickly guess how or why to use it without an example. > > > > > Thanks for the review Ben: > > I can move that logic to command-line.c to new function in ovs repo > > as ovs_cmdl_env_parse_all and use it in nbctl and sbctl. Will update > > example in doc accordingly in v2. > > Thanks! > Thanks Ben. Sent https://patchwork.ozlabs.org/patch/1177492/ and https://patchwork.ozlabs.org/patch/1177492/ to handle the same. PTAL. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev