Skip to content

Commit

Permalink
Fix ELB target integration test (#2078)
Browse files Browse the repository at this point in the history
Fix ELB target integration test

SUMMARY
follow up to ansible-collections/amazon.aws#2040
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
elb_target
ADDITIONAL INFORMATION
  • Loading branch information
tremble authored Apr 14, 2024
1 parent b5d4ea7 commit 2e8289f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
4 changes: 1 addition & 3 deletions plugins/modules/glue_connection.py
Original file line number Diff line number Diff line change
Expand Up @@ -298,9 +298,7 @@ def create_or_update_glue_connection(connection, connection_ec2, module, glue_co
params["ConnectionInput"]["PhysicalConnectionRequirements"] = dict()
if module.params.get("security_groups") is not None:
# Get security group IDs from names
security_group_ids = get_ec2_security_group_ids_from_names(
module.params.get("security_groups"), connection_ec2
)
security_group_ids = get_ec2_security_group_ids_from_names(module.params.get("security_groups"), connection_ec2)
params["ConnectionInput"]["PhysicalConnectionRequirements"]["SecurityGroupIdList"] = security_group_ids
if module.params.get("subnet_id") is not None:
params["ConnectionInput"]["PhysicalConnectionRequirements"]["SubnetId"] = module.params.get("subnet_id")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
state: present
zip_file: /tmp/lambda.zip
runtime: python3.12
role: "{{ ROLE_ARN.arn }}"
role: "{{ ROLE_ARN.iam_role.arn }}"
handler: ansible_lambda_target.lambda_handler
timeout: 30
register: lambda_function
Expand Down

0 comments on commit 2e8289f

Please sign in to comment.