From 2eb50032efe1aef669dcd79ab4d29f172031dab4 Mon Sep 17 00:00:00 2001 From: Markus Bergholz Date: Wed, 1 Jun 2022 13:12:03 +0200 Subject: [PATCH] ecs_service and ecs_service_info: add name and service aliases (#1187) ecs_service and ecs_service_info: add name and service aliases SUMMARY while ecs_service is using name for the service name parameter, ecs_service_info is using service for the same purpose. this PR adds just aliases to both modules, to use the same parameter to address the ecs service name. ref #1142 ISSUE TYPE Bugfix Pull Request Docs Pull Request Feature Pull Request COMPONENT NAME ecs_service ecs_service_info Reviewed-by: Alina Buzachis Reviewed-by: Joseph Torcasso Reviewed-by: Mark Chappell --- ecs_service.py | 3 ++- ecs_service_info.py | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/ecs_service.py b/ecs_service.py index 4f94a452ce9..9f35c26b9fa 100644 --- a/ecs_service.py +++ b/ecs_service.py @@ -34,6 +34,7 @@ - The name of the service. required: true type: str + aliases: ['service'] cluster: description: - The name of the cluster in which the service exists. @@ -662,7 +663,7 @@ def health_check_setable(self, params): def main(): argument_spec = dict( state=dict(required=True, choices=['present', 'absent', 'deleting']), - name=dict(required=True, type='str'), + name=dict(required=True, type='str', aliases=['service']), cluster=dict(required=False, type='str'), task_definition=dict(required=False, type='str'), load_balancers=dict(required=False, default=[], type='list', elements='dict'), diff --git a/ecs_service_info.py b/ecs_service_info.py index e6167afd09c..b04f94241f5 100644 --- a/ecs_service_info.py +++ b/ecs_service_info.py @@ -40,6 +40,7 @@ required: false type: list elements: str + aliases: ['name'] extends_documentation_fragment: - amazon.aws.aws - amazon.aws.ec2 @@ -214,7 +215,7 @@ def main(): details=dict(type='bool', default=False), events=dict(type='bool', default=True), cluster=dict(), - service=dict(type='list', elements='str') + service=dict(type='list', elements='str', aliases=['name']) ) module = AnsibleAWSModule(argument_spec=argument_spec, supports_check_mode=True)