diff --git a/library/decort_account_info.py b/library/decort_account_info.py index da2460e..27713b2 100644 --- a/library/decort_account_info.py +++ b/library/decort_account_info.py @@ -141,7 +141,7 @@ class DecortAccountInfo(DecortController): ), type=dict( type='str', - choices=['B', 'D'], + choices=self.DISK_TYPES, ), ), ), diff --git a/module_utils/decort_utils.py b/module_utils/decort_utils.py index ebbab55..f03709a 100644 --- a/module_utils/decort_utils.py +++ b/module_utils/decort_utils.py @@ -121,6 +121,8 @@ class DecortController(object): 'STOPPING', ] + DISK_TYPES = ['B', 'D'] + RESOURCE_GROUP_STATUSES = [ 'CREATED', 'DELETED', @@ -2448,6 +2450,15 @@ class DecortController(object): `/cloudapi/account/listDisks`. """ + if disk_type and ( + not disk_type in self.DISK_TYPES + ): + self.result['msg'] = ( + f'{disk_type} is not valid disk type' + f' for filtering account disk list.' + ) + self.amodule.fail_json(**self.result) + sort_by = None if sort_by_field: if not sort_by_field in self.FIELDS_FOR_SORTING_ACCOUNT_DISK_LIST: