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 #1601/316f0acb backport][stable-5] ecs_service supports constraints and strategy update #1650

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jan 11, 2023

This is a backport of PR #1601 as merged into main (316f0ac).

SUMMARY
ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ecs_service

ADDITIONAL INFORMATION

related issue #1414


ecs_service supports constraints and strategy update

SUMMARY

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

ecs_service
ADDITIONAL INFORMATION

related issue #1414

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: 周丰 <[email protected]>
Reviewed-by: Alina Buzachis <None>
(cherry picked from commit 316f0ac)
@ansibullbot
Copy link

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

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 43s
✔️ build-ansible-collection SUCCESS in 5m 15s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 11s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 18s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 15m 14s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 12m 13s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 11m 07s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 38s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 8m 05s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 53s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 27s
✔️ ansible-test-changelog SUCCESS in 2m 51s
✔️ ansible-test-splitter SUCCESS in 2m 56s
✔️ integration-community.aws-1 SUCCESS in 5m 09s
⚠️ 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 added the mergeit Merge the PR (SoftwareFactory) label Jan 12, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 27s
✔️ build-ansible-collection SUCCESS in 5m 11s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 59s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 07s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 12m 06s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 11m 11s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 51s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 41s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 42s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 50s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 7m 05s
✔️ ansible-test-changelog SUCCESS in 2m 15s
✔️ ansible-test-splitter SUCCESS in 3m 14s
✔️ integration-community.aws-1 SUCCESS in 8m 46s
⚠️ 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 79d42a8 into stable-5 Jan 12, 2023
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-5/316f0acb76d81a3065aa78c4f37fc5ef87212bf6/pr-1601 branch January 12, 2023 16:17
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…ce_attribute (ansible-collections#1650)

ec2_instance: add AWSRetry.jittered_backoff on client.describe_instance_attribute

SUMMARY
try to fix InvalidInstanceID.NotFound

Could not describe attribute groupSet for instance i-…: An error occurred (InvalidInstanceID.NotFound) when calling the DescribeInstanceAttribute operation: The instance ID 'i-…' does not exist

The instance actually exists, it's just an API problem.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ec2_instance

Reviewed-by: Helen Bailey <[email protected]>
Reviewed-by: Alina Buzachis
Reviewed-by: tchernomax
Reviewed-by: Mark Chappell
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 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