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

Backport: elb_application_lb - check_mode support, alb attributes, vpc default sg #1025

Conversation

jatorcasso
Copy link
Contributor

SUMMARY

Backport #894 #963 #971 manually to resolve conflicts.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

elb_application_lb

…nsible-collections#894)

add check_mode for elb_application_lb* & refactor integration tests

SUMMARY
Add check_mode support for elb_application_lb* & refactor integration tests.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
elb_application_lb
elb_application_lb_info

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Joseph Torcasso <None>
Reviewed-by: Jill R <None>
Reviewed-by: Mark Woolley <[email protected]>
(cherry picked from commit 239136b)
elb_application_lb - support alb attributes

SUMMARY

Support modifying different alb specific attributes
Fixes ansible-collections#571
Depends-On ansible-collections/amazon.aws#696

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
elb_application_lb

Reviewed-by: Jill R <None>
Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Joseph Torcasso <None>
(cherry picked from commit 85bfce4)
…le-collections#971)

elb_application_lb - treat empty security group as VPC default

SUMMARY

Fixes idempotency issue when security_groups = [] by treating [] as using the VPC's default security group (like it does on creation).
Fixes ansible-collections#28
Used same logic as amazon.aws.ec2_vpc_route_table does for using default igw
Added integration tests

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
elb_application_lb

Reviewed-by: Jill R <None>
Reviewed-by: Mark Woolley <[email protected]>
(cherry picked from commit 20b726a)
@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request integration tests/integration module module needs_triage plugins plugin (any type) tests tests labels Mar 28, 2022
@markuman markuman added the mergeit Merge the PR (SoftwareFactory) label Mar 29, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit aac1f34 into ansible-collections:stable-3 Mar 29, 2022
@jatorcasso jatorcasso deleted the backports/stable-3/elb_application_lb branch March 29, 2022 15:50
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…#1025)

Cleanup after 4.2.0 Release

SUMMARY
With 4.2.0 tagged we now need to forward-port the changelog updates from 4.2.0
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
CHANGELOG.rst
changelogs/changelog.yaml
changelogs/fragments/
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
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 plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants