NetApp cDOT driver qos policy same name
Don't try to rename QoS Policy Group to the name it already has. This is an unnecessary operation, that fails on NetApp Api with "Reason - 18339:Policy with new name already exists". Closes-Bug: #1778761 Change-Id: Iad58131ca6b675a4577888e287cd4899ea257771
This commit is contained in:
parent
97daab4b3d
commit
5da7d76d8a
|
@ -3770,6 +3770,8 @@ class NetAppCmodeClient(client_base.NetAppBaseClient):
|
|||
@na_utils.trace
|
||||
def qos_policy_group_rename(self, qos_policy_group_name, new_name):
|
||||
"""Renames a QoS policy group."""
|
||||
if qos_policy_group_name == new_name:
|
||||
return
|
||||
api_args = {
|
||||
'policy-group-name': qos_policy_group_name,
|
||||
'new-name': new_name,
|
||||
|
|
|
@ -6415,6 +6415,15 @@ class NetAppClientCmodeTestCase(test.TestCase):
|
|||
self.client.send_request.assert_called_once_with(
|
||||
'qos-policy-group-rename', qos_policy_group_rename_args, False)
|
||||
|
||||
def test_qos_policy_group_rename_noop(self):
|
||||
self.mock_object(self.client, 'send_request')
|
||||
|
||||
# rename to same name = no-op
|
||||
self.client.qos_policy_group_rename(
|
||||
fake.QOS_POLICY_GROUP_NAME, fake.QOS_POLICY_GROUP_NAME)
|
||||
|
||||
self.assertFalse(self.client.send_request.called)
|
||||
|
||||
def test_mark_qos_policy_group_for_deletion_rename_failure(self):
|
||||
self.mock_object(self.client, 'qos_policy_group_exists',
|
||||
mock.Mock(return_value=True))
|
||||
|
|
Loading…
Reference in New Issue