-
Notifications
You must be signed in to change notification settings - Fork 44
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
introduce ansible-test-splitter job #978
Merged
ansible-zuul
merged 1 commit into
ansible:master
from
goneri:introduce-ansible-test-splitter-job_1234
Jul 2, 2021
Merged
introduce ansible-test-splitter job #978
ansible-zuul
merged 1 commit into
ansible:master
from
goneri:introduce-ansible-test-splitter-job_1234
Jul 2, 2021
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
goneri
added a commit
to goneri/amazon.aws
that referenced
this pull request
Jun 30, 2021
This was referenced Jun 30, 2021
ansible-zuul bot
added a commit
to ansible-collections/community.aws
that referenced
this pull request
Jul 2, 2021
Cleanup integration tests post-Zuul migration SUMMARY This collection has been operating on Zuul CI for some weeks now. Remove shippable references Use shorter unique_ids were resource_prefixes now exceed AWS resource length limits (zuul hostnames are longer than shippable was) Cleanup for CI bugs surfaced in the course of this PR ISSUE TYPE Bugfix Pull Request COMPONENT NAME shippable.yml tests/ README.md NOT-Depends-On: ansible/ansible-zuul-jobs#978 Reviewed-by: Mark Chappell <None> Reviewed-by: Jill R <None>
This commit changes the way we split up the AWS jobs. We've got now a first job called `ansible-test-splitter` that is in charge of: - identify what target needs to be run (like the `ansible-test` role) - split up the workload on different jobs This logic is handled by a new role called `ansible-test-splitter` and these two tasks are done through two Python scripts. Why Python? Well, the two tasks were getting overcomplicated and the use of Python simplify a lot the troubleshoot and the code complexity. In addition, we can later create action plugin from these scripts. Finally, the commit also introduces a semaphore for the AWS jobs. It's a way to control the number of paralel jobs.
alinabuzachis
pushed a commit
to alinabuzachis/community.aws
that referenced
this pull request
Jul 15, 2021
…e_shippable Cleanup integration tests post-Zuul migration SUMMARY This collection has been operating on Zuul CI for some weeks now. Remove shippable references Use shorter unique_ids were resource_prefixes now exceed AWS resource length limits (zuul hostnames are longer than shippable was) Cleanup for CI bugs surfaced in the course of this PR ISSUE TYPE Bugfix Pull Request COMPONENT NAME shippable.yml tests/ README.md NOT-Depends-On: ansible/ansible-zuul-jobs#978 Reviewed-by: Mark Chappell <None> Reviewed-by: Jill R <None> This commit was initially merged in https://github.com/ansible-collections/community.aws See: ansible-collections@bd1008c
alinabuzachis
pushed a commit
to alinabuzachis/community.aws
that referenced
this pull request
Jul 17, 2021
…e_shippable Cleanup integration tests post-Zuul migration SUMMARY This collection has been operating on Zuul CI for some weeks now. Remove shippable references Use shorter unique_ids were resource_prefixes now exceed AWS resource length limits (zuul hostnames are longer than shippable was) Cleanup for CI bugs surfaced in the course of this PR ISSUE TYPE Bugfix Pull Request COMPONENT NAME shippable.yml tests/ README.md NOT-Depends-On: ansible/ansible-zuul-jobs#978 Reviewed-by: Mark Chappell <None> Reviewed-by: Jill R <None> This commit was initially merged in https://github.com/ansible-collections/community.aws See: ansible-collections@94be338
alinabuzachis
pushed a commit
to alinabuzachis/community.aws
that referenced
this pull request
Jul 19, 2021
Cleanup integration tests post-Zuul migration SUMMARY This collection has been operating on Zuul CI for some weeks now. Remove shippable references Use shorter unique_ids were resource_prefixes now exceed AWS resource length limits (zuul hostnames are longer than shippable was) Cleanup for CI bugs surfaced in the course of this PR ISSUE TYPE Bugfix Pull Request COMPONENT NAME shippable.yml tests/ README.md NOT-Depends-On: ansible/ansible-zuul-jobs#978 Reviewed-by: Mark Chappell <None> Reviewed-by: Jill R <None>
alinabuzachis
pushed a commit
to alinabuzachis/community.aws
that referenced
this pull request
Jul 19, 2021
Cleanup integration tests post-Zuul migration SUMMARY This collection has been operating on Zuul CI for some weeks now. Remove shippable references Use shorter unique_ids were resource_prefixes now exceed AWS resource length limits (zuul hostnames are longer than shippable was) Cleanup for CI bugs surfaced in the course of this PR ISSUE TYPE Bugfix Pull Request COMPONENT NAME shippable.yml tests/ README.md NOT-Depends-On: ansible/ansible-zuul-jobs#978 Reviewed-by: Mark Chappell <None> Reviewed-by: Jill R <None>
alinabuzachis
pushed a commit
to alinabuzachis/community.aws
that referenced
this pull request
Jul 19, 2021
Cleanup integration tests post-Zuul migration SUMMARY This collection has been operating on Zuul CI for some weeks now. Remove shippable references Use shorter unique_ids were resource_prefixes now exceed AWS resource length limits (zuul hostnames are longer than shippable was) Cleanup for CI bugs surfaced in the course of this PR ISSUE TYPE Bugfix Pull Request COMPONENT NAME shippable.yml tests/ README.md NOT-Depends-On: ansible/ansible-zuul-jobs#978 Reviewed-by: Mark Chappell <None> Reviewed-by: Jill R <None>
jillr
added a commit
to jillr/ansible-zuul-jobs
that referenced
this pull request
Jul 30, 2021
This reverts commit c3d89eb.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The goal of the job is to split the work load and prepare a list of targets to run fo the integration jobs.