diff --git a/lib/client/gnt_group.py b/lib/client/gnt_group.py index d3041ab279d3327590c1048b05c16f301f160fea..9ec8cd408ea8d839c2576e1b6232d39027ff330c 100644 --- a/lib/client/gnt_group.py +++ b/lib/client/gnt_group.py @@ -63,7 +63,7 @@ def AssignNodes(opts, args): group_name = args[0] node_names = args[1:] - op = opcodes.OpAssignGroupNodes(group_name=group_name, nodes=node_names, + op = opcodes.OpGroupAssignNodes(group_name=group_name, nodes=node_names, force=opts.force) SubmitOpCode(op, opts=opts) diff --git a/lib/cmdlib.py b/lib/cmdlib.py index ecf0bc9ec5d9de493b7b6acfac4fe296513c0ed2..c5cba850c85edcf0509c64158fc5d7f35d83bb8e 100644 --- a/lib/cmdlib.py +++ b/lib/cmdlib.py @@ -9975,7 +9975,7 @@ class LUGroupAdd(LogicalUnit): del self.remove_locks[locking.LEVEL_NODEGROUP] -class LUAssignGroupNodes(NoHooksLU): +class LUGroupAssignNodes(NoHooksLU): """Logical unit for assigning nodes to groups. """ diff --git a/lib/opcodes.py b/lib/opcodes.py index 4f6ed28dad251819185ab71f984972bd0a7cdc51..80d0f7c5ba49b9a0c1f8b10ef6129449d76cf488 100644 --- a/lib/opcodes.py +++ b/lib/opcodes.py @@ -1027,7 +1027,7 @@ class OpGroupAdd(OpCode): ] -class OpAssignGroupNodes(OpCode): +class OpGroupAssignNodes(OpCode): """Assign nodes to a node group.""" OP_ID = "OP_GROUP_ASSIGN_NODES" OP_DSC_FIELD = "group_name" diff --git a/lib/rapi/rlib2.py b/lib/rapi/rlib2.py index d1bd1c0c97babb84094935be0046b6dede2b30a2..4c378dbf03316f8ac807a98ec1aacf86be1a5514 100644 --- a/lib/rapi/rlib2.py +++ b/lib/rapi/rlib2.py @@ -695,7 +695,7 @@ class R_2_groups_name_assign_nodes(baserlib.R_Generic): @return: a job id """ - op = baserlib.FillOpcode(opcodes.OpAssignGroupNodes, self.request_body, { + op = baserlib.FillOpcode(opcodes.OpGroupAssignNodes, self.request_body, { "group_name": self.items[0], "dry_run": self.dryRun(), "force": self.useForce(), diff --git a/test/ganeti.cmdlib_unittest.py b/test/ganeti.cmdlib_unittest.py index c96745804bf11e45b9d6838af700a77a14bdea99..65a08fed9a5e8487ae99a13bf37fce82069f611f 100755 --- a/test/ganeti.cmdlib_unittest.py +++ b/test/ganeti.cmdlib_unittest.py @@ -157,7 +157,7 @@ class TestLUQuery(unittest.TestCase): "xyz") -class TestLUAssignGroupNodes(unittest.TestCase): +class TestLUGroupAssignNodes(unittest.TestCase): def testCheckAssignmentForSplitInstances(self): node_data = dict((name, objects.Node(name=name, group=group)) @@ -190,7 +190,7 @@ class TestLUAssignGroupNodes(unittest.TestCase): # Test first with the existing state. (new, prev) = \ - cmdlib.LUAssignGroupNodes.CheckAssignmentForSplitInstances([], + cmdlib.LUGroupAssignNodes.CheckAssignmentForSplitInstances([], node_data, instance_data) @@ -199,7 +199,7 @@ class TestLUAssignGroupNodes(unittest.TestCase): # And now some changes. (new, prev) = \ - cmdlib.LUAssignGroupNodes.CheckAssignmentForSplitInstances([("n1b", + cmdlib.LUGroupAssignNodes.CheckAssignmentForSplitInstances([("n1b", "g3")], node_data, instance_data)