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-3: ecs_service deployment_circuit_breaker #1225

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 5m 41s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 39s
✔️ ansible-test-sanity-docker-devel SUCCESS in 12m 14s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 32s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 11m 57s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 10m 12s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 53s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 5m 44s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 13s
✔️ ansible-test-splitter SUCCESS in 3m 06s
✔️ integration-community.aws-1 SUCCESS in 5m 43s
⚠️ 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 c630be0 into ansible-collections:stable-3 Jun 7, 2022
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…e reuse (ansible-collections#1227)

Move botocore version validation into module_utils.botocore for future reuse

Depends-On: ansible-collections#1246
SUMMARY
In preparation for ansible-collections#1225 move the SDK version validation pieces over into module_utils.botocore
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/module_utils/botocore.py
plugins/module_utils/core.py
plugins/module_utils/exceptions.py
plugins/module_utils/modules.py
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…collections#1248)

Refacter docs fragments and synchronize modules and plugins

SUMMARY
Move some of our docs fragments about to better reflect what they're for and the fact that we now have our own "amazon.aws..." namespace we can use.

Synchronises the modules and plugins docs fragments (as far as possible)
Updates plugin docs fragments to reflect the previously announced env variable deprecations

Note: the amazon.aws.common.plugins fragment will necessitate some changes to the inventory and lookup plugins before use.  (See also ansible-collections#1225).  However, those changes will then bring them in line with the other modules
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
plugins/docs_fragments
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
Refacter lookup plugins

Depends-On: ansible-collections#1248
SUMMARY

Refacters the lookup plugins to use common code for common boto3/botocore operations

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/lookup/aws_account_attribute.py
plugins/lookup/aws_secret.py
plugins/lookup/aws_ssm.py
plugins/module_utils/botocore.py
plugins/module_utils/core.py
plugins/module_utils/exceptions.py
plugins/module_utils/modules.py
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <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