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

elb_application_lb - treat empty security group as VPC default #971

Conversation

jatorcasso
Copy link
Contributor

SUMMARY
ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

elb_application_lb

@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review integration tests/integration module module needs_triage plugins plugin (any type) tests tests labels Mar 8, 2022
@jillr jillr added the mergeit Merge the PR (SoftwareFactory) label Mar 14, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 20b726a into ansible-collections:main Mar 14, 2022
jatorcasso added a commit to jatorcasso/community.aws that referenced this pull request Mar 28, 2022
…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)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Mar 29, 2022
…c default sg (#1025)

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

SUMMARY
Backport #894 #963 #971 manually to resolve conflicts.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
elb_application_lb

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…collections#971)

ec2_security_group: Set rules->group_name's datatype as raw

Signed-off-by: GomathiselviS [email protected]
SUMMARY
rules-> group_name can accept values of type list and str.
ISSUE TYPE


Docs Pull Request

COMPONENT NAME

ec2_security_group.py
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug community_review 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.

elb_application_lb with empty security groups list behaves inconsistently on create/update
4 participants