LGTM, thanks
On Tue, Aug 20, 2013 at 12:42 PM, Thomas Thrainer <[email protected]>wrote: > osparams are now recognized as changes, so changing only those without > other changes works too now. > > Signed-off-by: Thomas Thrainer <[email protected]> > --- > lib/cmdlib/instance.py | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/lib/cmdlib/instance.py b/lib/cmdlib/instance.py > index 137f2af..ac27f35 100644 > --- a/lib/cmdlib/instance.py > +++ b/lib/cmdlib/instance.py > @@ -2349,8 +2349,8 @@ class LUInstanceSetParams(LogicalUnit): > def CheckArguments(self): > if not (self.op.nics or self.op.disks or self.op.disk_template or > self.op.hvparams or self.op.beparams or self.op.os_name or > - self.op.offline is not None or self.op.runtime_mem or > - self.op.pnode): > + self.op.osparams or self.op.offline is not None or > + self.op.runtime_mem or self.op.pnode): > raise errors.OpPrereqError("No changes submitted", > errors.ECODE_INVAL) > > if self.op.hvparams: > -- > 1.8.3 > >
