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 #543/e3c8302c backport][stable-4] ECS service - add tag+propagate_tags upon creation. #1317

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jul 6, 2022

This is a backport of PR #543 as merged into main (e3c8302).

SUMMARY

this PR is continuation of #242
this PR will enable the use of tags at creation time, in addition to propagate_tags.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

ecs_service

ADDITIONAL INFORMATION

this PR is continuation of #242
i cant contribute to his repo as i am not a maintainer so i created a new PR, if its not the right git flow please be gentle, its my first time :)


ECS service - add tag+propagate_tags upon creation.

SUMMARY

this PR is continuation of #242
this PR will enable the use of tags at creation time, in addition to propagate_tags.

ISSUE TYPE

Feature Pull Request

COMPONENT NAME

ecs_service
ADDITIONAL INFORMATION

this PR is continuation of #242
i cant contribute to his repo as i am not a maintainer so i created a new PR, if its not the right git flow please be gentle, its my first time :)

Reviewed-by: Mark Chappell <None>
Reviewed-by: None <None>
Reviewed-by: Alina Buzachis <None>
(cherry picked from commit e3c8302)
@github-actions
Copy link

github-actions bot commented Jul 6, 2022

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

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

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 40s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 17s
✔️ ansible-test-sanity-docker-devel SUCCESS in 14m 41s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 14m 59s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 14m 36s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 16m 22s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 5m 59s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 5m 54s
✔️ ansible-test-splitter SUCCESS in 2m 33s
✔️ integration-community.aws-1 SUCCESS in 6m 07s
⚠️ 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 Jul 6, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 26s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 02s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 24s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 12m 18s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 15s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 11m 31s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 30s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 5m 22s
✔️ ansible-test-splitter SUCCESS in 3m 29s
✔️ integration-community.aws-1 SUCCESS in 5m 22s
⚠️ 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 bbbdaf2 into stable-4 Jul 6, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-4/e3c8302c3d5357fd501bc0385dee32a5b8098d37/pr-543 branch July 6, 2022 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request integration tests/integration mergeit Merge the PR (SoftwareFactory) module module new_contributor Help guide this first time contributor plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants