[ovs-dev] [PATCH 2/2] Add wrapper scripts for *ctl commands

Ben Pfaff blp at ovn.org
Tue Aug 2 16:03:52 UTC 2016


On Tue, Aug 02, 2016 at 07:56:27AM -0400, Russell Bryant wrote:
> On Tue, Aug 2, 2016 at 12:20 AM, Ryan Moats <rmoats at us.ibm.com> wrote:
> 
> > This commit creates wrapper scripts for the *ctl commands to use
> > --dry-run for those that have them, and to allow for log level
> > setting via ovs-appctl without allowing full access to ovs-appctl.
> > Tests have been added to make sure that the wrapper scripts
> > don't actually do anything when asked to perform a write operation.
> >
> > Signed-off-by: Ryan Moats <rmoats at us.ibm.com>
> >
> 
> What's the motivation for all the new "read" scripts?  It seems a bit
> confusing to install all of these.  They're also not documented anywhere.

My assumption had been that we'd put the options into the tree and then
that the one-liner redirection scripts would be an IBM customization.
After all, they need to customize somehow anyway to hide the read/write
versions in some off-$PATH place.



More information about the dev mailing list