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

tests: increase the default max duration of some tests #1578

Conversation

goneri
Copy link
Member

@goneri goneri commented Nov 1, 2022

  • s3_lifecycle up to 17 minutes
  • secretsmanager_secret up to 32m

See: https://ansible.softwarefactory-project.io/zuul/buildset/478fdb5e4d4f494584686371ff992a3b

@ansibullbot
Copy link

@goneri: Greetings! Thanks for taking the time to open this pullrequest. In order for the community to handle your pullrequest effectively, we need a bit more information.

Here are the items we could not find in your description:

  • issue type

Please set the description of this pullrequest with this template:
https://raw.githubusercontent.com/ansible/ansible/devel/.github/PULL_REQUEST_TEMPLATE.md

click here for bot help

@ansibullbot
Copy link

@ansibullbot ansibullbot added integration tests/integration needs_info This issue requires further information. Please answer any outstanding questions needs_template This issue/PR has an incomplete description. Please fill in the proposed template correctly needs_triage plugins plugin (any type) tests tests labels Nov 1, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 7m 21s
✔️ build-ansible-collection SUCCESS in 6m 09s
ansible-test-sanity-docker-devel FAILURE in 12m 33s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 20s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 33s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 20m 49s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 15m 15s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 42s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 15s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 09s
✔️ ansible-test-splitter SUCCESS in 2m 38s
integration-community.aws-1 TIMED_OUT in 1h 01m 10s
⚠️ 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
ansible-test-changelog FAILURE in 4m 43s

@goneri goneri force-pushed the tests-increase-the-default-max-duration-of-some-tests_13848 branch from 56ded6b to a2f6380 Compare November 2, 2022 13:20
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 44s
✔️ build-ansible-collection SUCCESS in 4m 58s
✔️ ansible-test-sanity-docker-devel SUCCESS in 13m 23s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 38s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 17s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 30s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 55s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 25s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 11s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 50s
✔️ ansible-test-splitter SUCCESS in 2m 30s
✔️ integration-community.aws-1 SUCCESS in 5m 50s
✔️ integration-community.aws-2 SUCCESS in 24m 53s
⚠️ 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
ansible-test-changelog FAILURE in 2m 22s

@tremble tremble force-pushed the tests-increase-the-default-max-duration-of-some-tests_13848 branch from ee982a9 to e09a6c7 Compare November 3, 2022 14:39
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Nov 3, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 15s
✔️ build-ansible-collection SUCCESS in 5m 35s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 34s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 15m 29s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 59s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 38s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 00s
ansible-test-units-amazon-aws-python36 FAILURE in 7m 13s
ansible-test-units-amazon-aws-python38 FAILURE in 5m 51s
ansible-test-units-amazon-aws-python39 FAILURE in 5m 45s
✔️ ansible-test-splitter SUCCESS in 2m 34s
✔️ integration-community.aws-1 SUCCESS in 6m 29s
✔️ integration-community.aws-2 SUCCESS in 22m 25s
⚠️ 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
✔️ ansible-test-changelog SUCCESS in 2m 38s

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 14s
✔️ build-ansible-collection SUCCESS in 5m 38s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 39s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 08s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 59s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 8m 48s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 46s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 45s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 56s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 51s
✔️ ansible-test-splitter SUCCESS in 3m 00s
✔️ integration-community.aws-1 SUCCESS in 7m 53s
✔️ integration-community.aws-2 SUCCESS in 26m 20s
⚠️ 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
✔️ ansible-test-changelog SUCCESS in 2m 38s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 7aab6cf into ansible-collections:main Nov 3, 2022
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…le-collections#1578)

Remove unrelated snapshot comment from rds_instance state docs

SUMMARY
The state field for rds_instance mentions snapshots. But that's irrelevant. Probably an error when copy-pasting to create the module.
Fixes ansible-collections#1574
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
rds_instance

Reviewed-by: Mark Chappell
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
integration tests/integration mergeit Merge the PR (SoftwareFactory) needs_info This issue requires further information. Please answer any outstanding questions needs_template This issue/PR has an incomplete description. Please fill in the proposed template correctly plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants