-
Notifications
You must be signed in to change notification settings - Fork 342
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_classic_lb - failed to resolve security_group_ids when providing security_group_names #589
Closed
1 task done
Labels
Comments
@anjo-swe Thank you for raising this issue. Would you be willing to open a PR to fix this issue? |
ansible-zuul bot
pushed a commit
that referenced
this issue
Dec 21, 2021
…ity_group_names (#592) elb_classic_lb: fix - resolve security_group_ids when providing security_group_names SUMMARY Fix failed to resolve security_group_ids when providing security_group_names. Fix broken tasks in integration tests. Fixes #589. ISSUE TYPE Bugfix Pull Request COMPONENT NAME elb_classic_lb Reviewed-by: Mark Chappell <None> Reviewed-by: None <None>
patchback bot
pushed a commit
that referenced
this issue
Dec 21, 2021
…ity_group_names (#592) elb_classic_lb: fix - resolve security_group_ids when providing security_group_names SUMMARY Fix failed to resolve security_group_ids when providing security_group_names. Fix broken tasks in integration tests. Fixes #589. ISSUE TYPE Bugfix Pull Request COMPONENT NAME elb_classic_lb Reviewed-by: Mark Chappell <None> Reviewed-by: None <None> (cherry picked from commit a2be6da)
@mandar242 thanks for the quick turnaround! |
ansible-zuul bot
pushed a commit
that referenced
this issue
Jan 12, 2022
…ity_group_names (#592) (#594) [PR #592/a2be6da7 backport][stable-2] elb_classic_lb: fix - resolve security_group_ids when providing security_group_names This is a backport of PR #592 as merged into main (a2be6da). SUMMARY Fix failed to resolve security_group_ids when providing security_group_names. Fix broken tasks in integration tests. Fixes #589. ISSUE TYPE Bugfix Pull Request COMPONENT NAME elb_classic_lb Reviewed-by: Jill R <None> Reviewed-by: None <None>
This is still an issue. At least for |
1 task
softwarefactory-project-zuul bot
pushed a commit
that referenced
this issue
Jul 6, 2022
Fix NoneType errors with elb_classic_lb SUMMARY fixes: #589 fixes: #914 Fixes two NoneType related bugs when creating new ELBs. (includes extra tests this time to trigger the bugs) ISSUE TYPE Bugfix Pull Request COMPONENT NAME plugins/modules/elb_classic_lb.py ADDITIONAL INFORMATION Reviewed-by: Alina Buzachis <None>
patchback bot
pushed a commit
that referenced
this issue
Jul 6, 2022
Fix NoneType errors with elb_classic_lb SUMMARY fixes: #589 fixes: #914 Fixes two NoneType related bugs when creating new ELBs. (includes extra tests this time to trigger the bugs) ISSUE TYPE Bugfix Pull Request COMPONENT NAME plugins/modules/elb_classic_lb.py ADDITIONAL INFORMATION Reviewed-by: Alina Buzachis <None> (cherry picked from commit 8ce9c19)
patchback bot
pushed a commit
that referenced
this issue
Jul 6, 2022
Fix NoneType errors with elb_classic_lb SUMMARY fixes: #589 fixes: #914 Fixes two NoneType related bugs when creating new ELBs. (includes extra tests this time to trigger the bugs) ISSUE TYPE Bugfix Pull Request COMPONENT NAME plugins/modules/elb_classic_lb.py ADDITIONAL INFORMATION Reviewed-by: Alina Buzachis <None> (cherry picked from commit 8ce9c19)
patchback bot
pushed a commit
that referenced
this issue
Jul 6, 2022
Fix NoneType errors with elb_classic_lb SUMMARY fixes: #589 fixes: #914 Fixes two NoneType related bugs when creating new ELBs. (includes extra tests this time to trigger the bugs) ISSUE TYPE Bugfix Pull Request COMPONENT NAME plugins/modules/elb_classic_lb.py ADDITIONAL INFORMATION Reviewed-by: Alina Buzachis <None> (cherry picked from commit 8ce9c19)
softwarefactory-project-zuul bot
pushed a commit
that referenced
this issue
Jul 6, 2022
[PR #915/8ce9c198 backport][stable-3] Fix NoneType errors with elb_classic_lb This is a backport of PR #915 as merged into main (8ce9c19). SUMMARY fixes: #589 fixes: #914 Fixes two NoneType related bugs when creating new ELBs. (includes extra tests this time to trigger the bugs) ISSUE TYPE Bugfix Pull Request COMPONENT NAME plugins/modules/elb_classic_lb.py ADDITIONAL INFORMATION Reviewed-by: Mark Chappell <None>
softwarefactory-project-zuul bot
pushed a commit
that referenced
this issue
Jul 6, 2022
[PR #915/8ce9c198 backport][stable-2] Fix NoneType errors with elb_classic_lb This is a backport of PR #915 as merged into main (8ce9c19). SUMMARY fixes: #589 fixes: #914 Fixes two NoneType related bugs when creating new ELBs. (includes extra tests this time to trigger the bugs) ISSUE TYPE Bugfix Pull Request COMPONENT NAME plugins/modules/elb_classic_lb.py ADDITIONAL INFORMATION Reviewed-by: Mark Chappell <None>
softwarefactory-project-zuul bot
pushed a commit
that referenced
this issue
Jul 6, 2022
[PR #915/8ce9c198 backport][stable-4] Fix NoneType errors with elb_classic_lb This is a backport of PR #915 as merged into main (8ce9c19). SUMMARY fixes: #589 fixes: #914 Fixes two NoneType related bugs when creating new ELBs. (includes extra tests this time to trigger the bugs) ISSUE TYPE Bugfix Pull Request COMPONENT NAME plugins/modules/elb_classic_lb.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
Summary
When i try to create a classic ELB it fails to resolve
security_group_ids
based onsecurity_group_names
&subnets
Locally i fixed both issues by:
if self.elb.get('Subnets'):
toif self.elb is not None and self.elb.get('Subnets'):
vpc_id = vpc_ids.pop()
toreturn vpc_ids.pop()
Issue Type
Bug Report
Component Name
elb_classic_lb
Ansible Version
Collection Versions
AWS SDK versions
Configuration
OS / Environment
macOS 11.6
Steps to Reproduce
Expected Results
Classic ELB named
dev-service-name
inside a VPC with 2 SGs attachedActual Results
Error 1 (only happens when creating)
Error 2
Code of Conduct
The text was updated successfully, but these errors were encountered: