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/