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_classic_lb: fix - resolve security_group_ids when providing security_group_names #592

Conversation

mandar242
Copy link
Contributor

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

@tremble tremble added the backport-2 PR should be backported to the stable-2 branch label Dec 21, 2021
@tremble tremble added the gate label Dec 21, 2021
Copy link
Contributor

@ansible-zuul ansible-zuul bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@ansible-zuul ansible-zuul bot merged commit a2be6da into ansible-collections:main Dec 21, 2021
@patchback
Copy link

patchback bot commented Dec 21, 2021

Backport to stable-2: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-2/a2be6da7f4169527493ab2ec4ff061eb4d85ac7b/pr-592

Backported as #594

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request 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 mandar242 deleted the 589_elb_classic_lb_resolve_security_group_ids branch December 22, 2021 03:25
@mandar242 mandar242 restored the 589_elb_classic_lb_resolve_security_group_ids branch December 22, 2021 03:26
@mandar242 mandar242 deleted the 589_elb_classic_lb_resolve_security_group_ids branch December 26, 2021 16:39
ansible-zuul bot pushed a commit that referenced this pull request 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>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
Fix queue attribute comparison

SUMMARY

Fixes sqs queue attribute comparison so updates are not performed on subsequent runs.  Also adds integration tests to verify this behavior.

Fixes ansible-collections#578
ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

sqs_queue
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Mark Chappell <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
Fix queue attribute comparison

SUMMARY

Fixes sqs queue attribute comparison so updates are not performed on subsequent runs.  Also adds integration tests to verify this behavior.

Fixes ansible-collections#578
ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

sqs_queue
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Mark Chappell <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-2 PR should be backported to the stable-2 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

elb_classic_lb - failed to resolve security_group_ids when providing security_group_names
2 participants