On Tue, Aug 2, 2016 at 9:37 AM, Ryan Moats <rmo...@us.ibm.com> wrote:
> "dev" <dev-boun...@openvswitch.org> wrote on 08/02/2016 08:34:08 AM: > > > From: Ryan Moats/Omaha/IBM@IBMUS > > To: Russell Bryant <russ...@ovn.org> > > Cc: ovs dev <dev@openvswitch.org> > > Date: 08/02/2016 08:35 AM > > Subject: Re: [ovs-dev] [PATCH 2/2] Add wrapper scripts for *ctl commands > > Sent by: "dev" <dev-boun...@openvswitch.org> > > > > > > > > Russell Bryant <russ...@ovn.org> wrote on 08/02/2016 07:11:38 AM: > > > > > From: Russell Bryant <russ...@ovn.org> > > > To: Ryan Moats/Omaha/IBM@IBMUS > > > Cc: ovs dev <dev@openvswitch.org> > > > 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 <russ...@ovn.org> > wrote: > > > > > > On Tue, Aug 2, 2016 at 12:20 AM, Ryan Moats <rmo...@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 <rmo...@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. > > > > > > 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 dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
