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

[CI] Introduce concurrency in package testing #297

Closed
ycombinator opened this issue Oct 6, 2020 · 2 comments · Fixed by #421
Closed

[CI] Introduce concurrency in package testing #297

ycombinator opened this issue Oct 6, 2020 · 2 comments · Fixed by #421
Assignees
Labels
Team:Automation Label for the Observability productivity team

Comments

@ycombinator
Copy link
Contributor

Currently we run pipeline tests for all packages first, followed by system tests for all packages. As the number of packages grows, as we define pipeline and system tests for more packages, and when we add a third type of test (asset loading tests), this serial ordering will slow down CI jobs.

An initial optimization might be to run each type of test for all packages concurrently. So if we have N packages and M test types, there will be M concurrent sub-jobs, one for running each type of test on the N packages.

A further optimization might be to run each type of test (pipeline, system, etc.) for each package concurrently. So if we have N packages and M test types, there will be N x M concurrent sub-jobs, one for running each type of test on each package.

@andresrc andresrc added the Team:Automation Label for the Observability productivity team label Oct 7, 2020
@mtojek
Copy link
Contributor

mtojek commented Nov 17, 2020

Be aware that changes in elastic/package-spec#31 may not be thread-safe.

@ycombinator
Copy link
Contributor Author

Thanks @mtojek. My understanding is that the concurrency being described in this issue will result in multiple, concurrently-running processes so we shouldn't be affected by the non-thread-safe changes in elastic/package-spec#31. Still, it's good to be aware of the current limitations 👍.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Automation Label for the Observability productivity team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants