On Mon, Feb 14, 2011 at 5:37 PM, Michael Hanselmann <[email protected]> wrote: > --- > doc/rapi.rst | 4 ++-- > lib/rapi/rlib2.py | 8 ++++---- > 2 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/doc/rapi.rst b/doc/rapi.rst > index 316c8f1..940f35a 100644 > --- a/doc/rapi.rst > +++ b/doc/rapi.rst > @@ -469,8 +469,8 @@ Returns a job ID. > > Body parameters: > > -``new_name`` (string, required) > - New node group name. > +.. opcode_params:: OP_GROUP_RENAME > + :exclude: group_name > > > ``/2/groups/[group_name]/assign-nodes`` > diff --git a/lib/rapi/rlib2.py b/lib/rapi/rlib2.py > index 2aaa774..371faf7 100644 > --- a/lib/rapi/rlib2.py > +++ b/lib/rapi/rlib2.py > @@ -664,10 +664,10 @@ def _ParseRenameGroupRequest(name, data, dry_run): > @return: Node group rename opcode > > """ > - new_name = baserlib.CheckParameter(data, "new_name") > - > - return opcodes.OpGroupRename(group_name=name, new_name=new_name, > - dry_run=dry_run) > + return baserlib.FillOpcode(opcodes.OpGroupRename, data, { > + "group_name": name, > + "dry_run": dry_run, > + }) > > > class R_2_groups_name_rename(baserlib.R_Generic): > -- > 1.7.3.5
LGTM > >
