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

[PR #1212/bac24736 backport][stable-2] ecs_* - fix idempotence bug in ecs_service and dont require cluster #1230

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jun 8, 2022

This is a backport of PR #1212 as merged into main (bac2473).

SUMMARY
  • Don't require cluster param and use cluster name 'default' when not specified (see docs).
  • Fix bug when comparing health_check_grace_period_seconds when not input by user.
ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME
  • ecs_service
  • ecs_task
ADDITIONAL INFORMATION

Split up from #1209 to backport to stable-2

…`` (#1212)

ecs_* - fix idempotence bug in ecs_service and dont require ``cluster``

SUMMARY

Don't require cluster param and use cluster name 'default' when not specified (see docs).
Fix bug when comparing health_check_grace_period_seconds when not input by user.

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

ecs_service
ecs_task

ADDITIONAL INFORMATION
Split up from #1209 to backport to stable-2

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis <None>
(cherry picked from commit bac2473)
@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review module module needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) labels Jun 8, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

ansible-galaxy-importer FAILURE in 4m 17s (non-voting)
✔️ build-ansible-collection SUCCESS in 4m 51s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 28s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 52s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 11m 02s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 11m 28s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 17s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 7m 02s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 48s
✔️ ansible-test-splitter SUCCESS in 2m 24s
✔️ integration-community.aws-1 SUCCESS in 5m 16s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Jun 15, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

ansible-galaxy-importer FAILURE in 3m 41s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 21s
✔️ ansible-test-sanity-docker-devel SUCCESS in 8m 57s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 06s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 9m 54s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 56s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 11m 25s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 14s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 5m 56s
✔️ ansible-test-splitter SUCCESS in 2m 23s
✔️ integration-community.aws-1 SUCCESS in 5m 15s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit c43fb28 into stable-2 Jun 15, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-2/bac247365347a6b6e4f90c8087f235b01cf8d253/pr-1212 branch June 15, 2022 07:44
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…lections#1230)

Move RetryingBotoClientWrapper into module_utils.retries

Depends-On: ansible-collections#1579
SUMMARY
Move RetryingBotoClientWrapper into module_utils.retries
This means we can use it later with non-module plugins
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/module_utils/modules.py
plugins/module_utils/retries.py
plugins/module_utils/waiters.py
ADDITIONAL INFORMATION

Reviewed-by: Gonéri Le Bouder <[email protected]>
Reviewed-by: Alina Buzachis <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 mergeit Merge the PR (SoftwareFactory) module module new_contributor Help guide this first time contributor plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants