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

backport-2: ecs_service deployment_circuit_breaker #1226

Merged
merged 2 commits into from
Jun 7, 2022
Merged

Conversation

markuman
Copy link
Member

@markuman markuman commented Jun 7, 2022

SUMMARY

manual backport of

there is no version_added, because the module already supports this feature. it was just undocumented.

ISSUE TYPE
  • Bugfix Pull Request
  • Docs Pull Request
  • Feature Pull Request
COMPONENT NAME

ecs_service

ADDITIONAL INFORMATION

ecs_service - document circuit breaker feature

SUMMARY
Fixes #921
This feature works with the existing code, so this was mainly adding documentation, examples, and an integration test.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ecs_service
ADDITIONAL INFORMATION
The deployment circuit breaker is part of the deployment configuration dictionary, which is already snake<=>camel cased. Thus the existing code was handling 99% of the feature, we just added some type validation, documentation, examples, and an integration test.
- community.aws.ecs_service:
    state: present
    name: test-service
    cluster: test-cluster
    task_definition: test-task-definition
    desired_count: 3
    deployment_configuration:
      deployment_circuit_breaker:
        enable: True
        rollback: True

Reviewed-by: Mark Chappell <None>
@ansibullbot
Copy link

@markuman This PR contains @ mentions in at least one commit message. Those mentions can cause cascading notifications through GitHub and need to be removed. Please squash or amend your commits to remove the mentions.

click here for bot help

@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug integration tests/integration module module needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR needs_triage plugins plugin (any type) tests tests labels Jun 7, 2022
fix assert of deploymentCircuitBreaker

SUMMARY
fix broken ecs_cluster integration test of #1215
ISSUE TYPE

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell <None>
@ansibullbot ansibullbot added community_review and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Jun 7, 2022
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Jun 7, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

ansible-galaxy-importer FAILURE in 3m 55s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 09s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 52s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 12m 28s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 12m 53s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 9m 48s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 05s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 43s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 5m 25s
✔️ ansible-test-splitter SUCCESS in 2m 39s
✔️ integration-community.aws-1 SUCCESS in 5m 26s
⚠️ 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 f409bf3 into ansible-collections:stable-2 Jun 7, 2022
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
Fix non-matching defaults in docs

SUMMARY
Fix various non-matching default values exposed by ansible/ansible#79267.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
various

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 integration tests/integration mergeit Merge the PR (SoftwareFactory) module module needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants