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

ecs_cluster: add default value to doc #1676

Conversation

markuman
Copy link
Member

SUMMARY

missing default value in the docs section of #1640
not released yet.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

ecs_cluster

@github-actions
Copy link

github-actions bot commented Jan 23, 2023

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

@ansibullbot
Copy link

@softwarefactory-project-zuul

This comment was marked as resolved.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 9m 08s
✔️ build-ansible-collection SUCCESS in 6m 06s
✔️ ansible-test-sanity-docker-devel SUCCESS in 8m 42s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 8m 42s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 08s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 8m 50s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 44s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 5m 55s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 36s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 11s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 5m 56s
✔️ ansible-test-changelog SUCCESS in 2m 17s
✔️ ansible-test-splitter SUCCESS in 2m 49s
✔️ integration-community.aws-1 SUCCESS in 5m 27s
⚠️ 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
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@markuman markuman requested a review from tremble January 23, 2023 17:38
@tremble tremble added the backport-5 PR should be backported to the stable-5 branch label Jan 23, 2023
@markuman markuman added mergeit Merge the PR (SoftwareFactory) and removed backport-5 PR should be backported to the stable-5 branch labels Jan 23, 2023
@markuman
Copy link
Member Author

must be included in the manual backport #1673

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 43s
✔️ build-ansible-collection SUCCESS in 6m 11s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 04s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 00s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 18s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 8m 43s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 43s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 5m 49s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 32s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 57s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 5m 57s
✔️ ansible-test-changelog SUCCESS in 2m 47s
✔️ ansible-test-splitter SUCCESS in 3m 13s
✔️ 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
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit a11073e into ansible-collections:main Jan 23, 2023
markuman added a commit that referenced this pull request Jan 23, 2023
ecs_cluster: add default value to doc

SUMMARY
missing default value in the docs section of #1640
not released yet.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
ecs_cluster

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jan 23, 2023
… (#1678)

 backport stable-5 - ecs_cluster capacity provider strategy #1640 #1673 

backport #1640 ecs_cluster capacity provider strategy
#1640 (comment)
backport #1676
waiting for #1676
SUMMARY
Fixes #770 - Add AWS ECS_Cluster Capacity Provider Strategy Support ISSUE TYPE
Feature Pull Request
COMPONENT NAME
ecs_cluster
ADDITIONAL INFORMATION
When creating or updating an ECS Cluster, configure the capacity providers and capacity provider strategy as provided by the user. Given playbook task:

name: Create an ECS Cluster with Capacity Providers ecs_cluster: name: default state: present capacity_providers: - FARGATE - FARGATE_SPOT capacity_provider_strategy: - capacity_provider: FARGATE base: 1 weight: 1 - capacity_provider: FARGATE_SPOT weight: 100

Previously would throw "Unsupported parameter" and no other parameter exists to expose these features. Now you should see changed: [localhost] with the resultant created ECS Cluster having the same providers and provider_strategy fields as provided by the user.
Reviewed-by: Markus Bergholz [email protected]
Reviewed-by: Justin McCormick 
Reviewed-by: Alina Buzachis 
SUMMARY


ISSUE TYPE


Bugfix Pull Request
Docs Pull Request
Feature Pull Request
New Module Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review docs mergeit Merge the PR (SoftwareFactory) module module needs_triage owner_pr PR created by owner/maintainer plugins plugin (any type) small_patch Hopefully easy to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants