diff --git a/lib/client/gnt_group.py b/lib/client/gnt_group.py index 538c1cbe9fa4cf6ba6fd77844542d86e769bb459..2fb73e1b07b62e0194dc6e9da08154f7159bf0b5 100644 --- a/lib/client/gnt_group.py +++ b/lib/client/gnt_group.py @@ -130,17 +130,13 @@ def SetGroupParams(opts, args): @return: the desired exit code """ - all_changes = { - "ndparams": opts.ndparams, - "alloc_policy": opts.alloc_policy, - } - - if all_changes.values().count(None) == len(all_changes): + if opts.ndparams is None and opts.alloc_policy is None: ToStderr("Please give at least one of the parameters.") return 1 - op = opcodes.OpGroupSetParams(group_name=args[0], # pylint: disable-msg=W0142 - **all_changes) + op = opcodes.OpGroupSetParams(group_name=args[0], + ndparams=opts.ndparams, + alloc_policy=opts.alloc_policy) result = SubmitOrSend(op, opts) if result: diff --git a/lib/cmdlib.py b/lib/cmdlib.py index 80795d2abfd91453fb61cbb400f59a4effac41d5..5b1be95ce4e54757804c1a00998c40ec13002729 100644 --- a/lib/cmdlib.py +++ b/lib/cmdlib.py @@ -9831,6 +9831,8 @@ class TLReplaceDisks(Tasklet): """ steps_total = 6 + pnode = self.instance.primary_node + # Step: check device activation self.lu.LogStep(1, steps_total, "Check device existence") self._CheckDisksExistence([self.instance.primary_node]) @@ -9905,10 +9907,8 @@ class TLReplaceDisks(Tasklet): " soon as possible")) self.lu.LogInfo("Detaching primary drbds from the network (=> standalone)") - result = self.rpc.call_drbd_disconnect_net([self.instance.primary_node], - self.node_secondary_ip, - self.instance.disks)\ - [self.instance.primary_node] + result = self.rpc.call_drbd_disconnect_net([pnode], self.node_secondary_ip, + self.instance.disks)[pnode] msg = result.fail_msg if msg: