D'oh! To reply to myself:

On Fri, Dec 02, 2016 at 02:26:21PM +0000, Brian Foley wrote:
> Maybe add an else: print warning about removing an option that's not set.
> 
> > +
> > +      for osp in remove_osparams_private:
> > +        if osp in private_parms:
> > +          raise errors.OpPrereqError("Requested both removal and addition 
> > of "
> > +                                     "param %s" % osp)

Maybe s/param/private param/

Reply via email to