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 #1680/6f844249 backport][stable-5] ecs_service -- with force_new_deployment user can specify taskdef or not #1732

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Mar 2, 2023

This is a backport of PR #1680 as merged into main (6f84424).

SUMMARY

Fixes #1106
Support force_new_deployment without having to specify a task definition.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

ecs_service

ADDITIONAL INFORMATION

Previously task_definition was required when state was present; regardless of whether force_new_deployment was set or not.
Previous error was along the lines of "state is present but all of the following are missing: task_definition".
New behavior enforces either task_definition or force_new_deployment is set.
If both are provided, the user's task_definition will be sent through to boto.
If only task_definition is defined, original behavior resumes.
If only force_new_deployment is set, pull the taskDefinition from existing and pass it through to boto.

…not (#1680)

ecs_service -- with force_new_deployment user can specify taskdef or not

SUMMARY
Fixes #1106
Support force_new_deployment without having to specify a task definition.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ecs_service
ADDITIONAL INFORMATION
Previously task_definition was required when state was present; regardless of whether force_new_deployment was set or not.
Previous error was along the lines of "state is present but all of the following are missing: task_definition".
New behavior enforces either task_definition or force_new_deployment is set.
If both are provided, the user's task_definition will be sent through to boto.
If only task_definition is defined, original behavior resumes.
If only force_new_deployment is set, pull the taskDefinition from existing and pass it through to boto.

Reviewed-by: Alina Buzachis
Reviewed-by: Mark Chappell
Reviewed-by: Markus Bergholz <[email protected]>
(cherry picked from commit 6f84424)
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Mar 2, 2023
@github-actions
Copy link

github-actions bot commented Mar 2, 2023

Docs Build 📝

Thank you for contribution!✨

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

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/853c7374e9b64e25b361ad00f1392e86

✔️ ansible-galaxy-importer SUCCESS in 3m 42s
✔️ build-ansible-collection SUCCESS in 12m 51s
✔️ ansible-test-sanity-docker-devel SUCCESS in 8m 51s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 09s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 05s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 8m 51s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 8m 53s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 5m 35s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 03s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 06s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 5m 36s
✔️ ansible-test-changelog SUCCESS in 4m 19s
✔️ ansible-test-splitter SUCCESS in 4m 40s
✔️ integration-community.aws-1 SUCCESS in 20m 02s
Skipped 21 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 5ab6594 into stable-5 Mar 2, 2023
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-5/6f844249a91b2d86dfd2a9be1f843bcb90366b2d/pr-1680 branch March 2, 2023 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants