Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PR #913/55962ff2 backport][stable-3] Add deregistration_connection_termination to elb_target_group #926

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Feb 4, 2022

This is a backport of PR #913 as merged into main (55962ff).

SUMMARY

Adding support for the deregistration_connection_termination param in the elb_target_group module.

Along with this I've enabled and fixed up the integration tests.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

elb_target_group

ADDITIONAL INFORMATION

The API param is deregistration_delay.connection_termination.enabled

https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/elbv2.html#ElasticLoadBalancingv2.Client.describe_target_group_attributes

Add deregistration_connection_termination to elb_target_group

SUMMARY
Adding support for the deregistration_connection_termination param in the elb_target_group module.
Along with this I've enabled and fixed up the integration tests.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
elb_target_group
ADDITIONAL INFORMATION
The API param is deregistration_delay.connection_termination.enabled
https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/elbv2.html#ElasticLoadBalancingv2.Client.describe_target_group_attributes

Reviewed-by: Mark Woolley <[email protected]>
Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis <None>
(cherry picked from commit 55962ff)
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (third-party-check pipeline).

@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request integration tests/integration module module needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) tests tests labels Feb 4, 2022
@markuman markuman added the mergeit Merge the PR (SoftwareFactory) label Feb 4, 2022
@softwarefactory-project-zuul
Copy link
Contributor

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 0fde2d4 into stable-3 Feb 4, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-3/55962ff21023ffdc6ecc1f985503c63852b3b19b/pr-913 branch February 4, 2022 21:07
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…nsible-collections#926)

ec2_vpc_route_table: Add support for Route entry for Carrier Gateway

SUMMARY
Add support for VPC Carrear Gateways entry on route table.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME

module/ec2_vpc_route_table

ADDITIONAL INFORMATION
Support Carrier Gateway route on Route Table module using the same strategy of  Nat GW, discovering the prefix of resource name cagw-.
Not directly related, but testing in the same solution the cagw modules: ansible-collections#1353

ec2_carrier_gateway
ec2_carrier_gateway_info

Reviewed-by: Mark Chappell
Reviewed-by: Marco Braga
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request integration tests/integration mergeit Merge the PR (SoftwareFactory) module module new_contributor Help guide this first time contributor plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants