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

list_changed_targets: drop build_up_batches #1625

Conversation

goneri
Copy link
Collaborator

@goneri goneri commented Sep 13, 2022

Use build_up_batches_by_time() by default. By default, we assume a target
takes 180s.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-tox-linters SUCCESS in 4m 55s
✔️ ansible-tox-py38 SUCCESS in 5m 18s
✔️ ansible-tox-py39 SUCCESS in 5m 22s
✔️ ansible-tox-py310 SUCCESS in 5m 04s

Copy link
Collaborator

@abikouo abikouo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, Thanks

@goneri
Copy link
Collaborator Author

goneri commented Sep 14, 2022

Nice, Thanks

Merci Aubin :-).

goneri added a commit to goneri/community.aws that referenced this pull request Sep 14, 2022
goneri added a commit to goneri/community.aws that referenced this pull request Sep 14, 2022
goneri added a commit to goneri/community.aws that referenced this pull request Sep 14, 2022
@goneri goneri force-pushed the list_changed_targets-drop-build_up_batches_27441 branch from f9e4267 to 64ce538 Compare September 14, 2022 19:38
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-tox-linters SUCCESS in 4m 54s
✔️ ansible-tox-py38 SUCCESS in 5m 32s
✔️ ansible-tox-py39 SUCCESS in 5m 13s
✔️ ansible-tox-py310 SUCCESS in 5m 25s

goneri added a commit to goneri/community.aws that referenced this pull request Sep 14, 2022
Use `build_up_batches_by_time()` by default. By default, we assume a target
takes 180s.
We also try to stay below 45 minutes to give enough time to environment set-up.
@goneri goneri force-pushed the list_changed_targets-drop-build_up_batches_27441 branch from 64ce538 to 747d88e Compare September 14, 2022 19:57
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-tox-linters SUCCESS in 5m 57s
✔️ ansible-tox-py38 SUCCESS in 5m 18s
✔️ ansible-tox-py39 SUCCESS in 6m 16s
✔️ ansible-tox-py310 SUCCESS in 5m 06s

goneri added a commit to goneri/community.aws that referenced this pull request Sep 16, 2022
@goneri goneri added the mergeit Merge the PR (SoftwareFActory) label Sep 16, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-tox-linters SUCCESS in 7m 00s
✔️ ansible-tox-py38 SUCCESS in 7m 43s
✔️ ansible-tox-py39 SUCCESS in 7m 37s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit bcb195b into ansible:master Sep 16, 2022
softwarefactory-project-zuul bot pushed a commit to ansible-collections/amazon.aws that referenced this pull request Sep 16, 2022
tests: some ec2_instance targets are slower than normal

Depends-On: ansible/ansible-zuul-jobs#1625
Dedicate more time for the execution of some slow ec2_instance targets.
See: #978 (comment)
softwarefactory-project-zuul bot pushed a commit to ansible-collections/community.aws that referenced this pull request Sep 16, 2022
…#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: #1468
Depends-On: #1473

Reviewed-by: Mark Chappell <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 26, 2023
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 27, 2023
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 15, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 15, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 15, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 15, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 16, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Oct 18, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Oct 18, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
mandar242 pushed a commit to mandar242/amazon.aws that referenced this pull request Oct 23, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
mandar242 pushed a commit to mandar242/amazon.aws that referenced this pull request Oct 23, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
mandar242 pushed a commit to mandar242/amazon.aws that referenced this pull request Oct 23, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
mandar242 pushed a commit to mandar242/amazon.aws that referenced this pull request Oct 23, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
mandar242 pushed a commit to mandar242/amazon.aws that referenced this pull request Oct 23, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
mandar242 pushed a commit to mandar242/amazon.aws that referenced this pull request Oct 23, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
mandar242 pushed a commit to mandar242/amazon.aws that referenced this pull request Oct 24, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
mandar242 pushed a commit to mandar242/amazon.aws that referenced this pull request Oct 24, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
mandar242 pushed a commit to mandar242/amazon.aws that referenced this pull request Oct 24, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Oct 24, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Oct 24, 2024
…ansible-collections#1467)

tests/integration: ensure the CI gives more time to some slow targets

Depends-On: ansible/ansible-zuul-jobs#1625
Depends-On: ansible-collections#1468
Depends-On: ansible-collections#1473

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@19a24e5
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