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

Fix queue attribute comparison #592

Merged
merged 2 commits into from
Jul 9, 2021

Conversation

msven
Copy link
Contributor

@msven msven commented Jun 3, 2021

SUMMARY

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

Fixes #578

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

sqs_queue

ADDITIONAL INFORMATION

@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 Jun 3, 2021
@msven
Copy link
Contributor Author

msven commented Jun 9, 2021

Seems like one of the checks is stuck. Not sure how to trigger it...

@tremble
Copy link
Contributor

tremble commented Jun 9, 2021

Don't worry about the "gate" check, this one will run once the PR has been reviewed and approved for merging (it's a final check that's run to make sure that the tests still pass before automatically merging the PR)

Copy link
Member

@markuman markuman left a comment

Choose a reason for hiding this comment

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

LGTM

@ansibullbot
Copy link

@tremble tremble added the gate label Jul 9, 2021
@ansible-zuul ansible-zuul bot merged commit 3513a09 into ansible-collections:main Jul 9, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 15, 2021
…eueIdempotent

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>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@3513a09
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 16, 2021
…eueIdempotent

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>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@3513a09
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 17, 2021
…eueIdempotent

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>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@36b1d6f
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
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>
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
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>
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
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>
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
…ity_group_names (ansible-collections#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 ansible-collections#589.

ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

elb_classic_lb

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

Successfully merging this pull request may close these issues.

sqs_queue is not idempotent when any queue attribute parameter passed in playbook
4 participants