On Tue, Aug 2, 2016 at 9:37 AM, Ryan Moats <[email protected]> wrote:
> "dev" <[email protected]> wrote on 08/02/2016 08:34:08 AM: > > > From: Ryan Moats/Omaha/IBM@IBMUS > > To: Russell Bryant <[email protected]> > > Cc: ovs dev <[email protected]> > > Date: 08/02/2016 08:35 AM > > Subject: Re: [ovs-dev] [PATCH 2/2] Add wrapper scripts for *ctl commands > > Sent by: "dev" <[email protected]> > > > > > > > > Russell Bryant <[email protected]> wrote on 08/02/2016 07:11:38 AM: > > > > > From: Russell Bryant <[email protected]> > > > To: Ryan Moats/Omaha/IBM@IBMUS > > > Cc: ovs dev <[email protected]> > > > Date: 08/02/2016 07:12 AM > > > Subject: Re: [ovs-dev] [PATCH 2/2] Add wrapper scripts for *ctl > commands > > > > > > On Tue, Aug 2, 2016 at 7:56 AM, Russell Bryant <[email protected]> > wrote: > > > > > > On Tue, Aug 2, 2016 at 12:20 AM, Ryan Moats <[email protected]> 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 <[email protected]> > > > > > > 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. > > > > > > I see the thread discussing this now. I'm still not a big fan of > > > installing this for everyone... > > > > I admit that I need to respin with better documentation, but I > > doubt that alone would make you a fan... > Right. > Meh, I hit send too soon. I should add that (as Mestery can tell you) > I'm not a real big fan of this either - it's a necessity... > The scripts seem small enough. I'm sure you're having to do deployment customization to ensure the scripts have the right access so your admins can use those, but not the full commands. Perhaps just include the scripts in that customization as well? -- Russell Bryant _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
