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.

Sign up for GitHub

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

Jump to bottom

Introduce component groups - let's see if this speeds up CI #8752

Open
wants to merge 13 commits into
base: development
Choose a base branch
from
Open

Introduce component groups - let's see if this speeds up CI #8752

wants to merge 13 commits into from

Conversation

tom-cosgrove-arm
Copy link
Contributor

PR checklist

Please tick as appropriate and edit the reasons (e.g.: "backport: not needed because this is a new feature")

  • changelog provided, or not required
  • backport done, or not required
  • tests provided, or not required

@tom-cosgrove-arm tom-cosgrove-arm added DO-NOT-MERGE size-s Estimated task size: small (~2d) component-test Test framework and CI scripts priority-low Low priority - this may not receive review soon labels Jan 25, 2024
@tom-cosgrove-arm tom-cosgrove-arm changed the title Introduce component groups - lets see if this speeds up CI Introduce component groups - let's see if this speeds up CI Jan 26, 2024
@daverodgman daverodgman self-requested a review January 26, 2024 10:43
@tom-cosgrove-arm
Introduce component groups - lets see if this speeds up CI
c442118 
Signed-off-by: Tom Cosgrove <tom.cosgrove@arm.com>
@tom-cosgrove-arm
Revert previous commit to be able to compare timings
8aed92a 
Signed-off-by: Tom Cosgrove <tom.cosgrove@arm.com>
@tom-cosgrove-arm
Introduce component groups - lets see if this speeds up CI
e26ee82 
Signed-off-by: Tom Cosgrove <tom.cosgrove@arm.com>
@tom-cosgrove-arm
Add some more component groups, based on CI results
83f128a 
Signed-off-by: Tom Cosgrove <tom.cosgrove@arm.com>
@tom-cosgrove-arm
More groups, again from CI results
9c007bc 
Signed-off-by: Tom Cosgrove <tom.cosgrove@arm.com>
@tom-cosgrove-arm
Don't include components with support_ functions in groups
27dee0c 
Signed-off-by: Tom Cosgrove <tom.cosgrove@arm.com>
@tom-cosgrove-arm
Try running the slowest components before anything else
ec14ca3 
Signed-off-by: Tom Cosgrove <tom.cosgrove@arm.com>
@tom-cosgrove-arm
Revert previous commit to be able to compare timings
d4f3842 
Signed-off-by: Tom Cosgrove <tom.cosgrove@arm.com>
@tom-cosgrove-arm
Try running the slowest components before anything else
0465757 
Signed-off-by: Tom Cosgrove <tom.cosgrove@arm.com>
@tom-cosgrove-arm
Revert previous commit to be able to compare timings
e71054f 
Signed-off-by: Tom Cosgrove <tom.cosgrove@arm.com>
@tom-cosgrove-arm
Revert all of this to be able to compare timings
d853961 
Signed-off-by: Tom Cosgrove <tom.cosgrove@arm.com>
@tom-cosgrove-arm
Revert previous two commits to get back to groups and slowest first
17aba7d 
Signed-off-by: Tom Cosgrove <tom.cosgrove@arm.com>
@tom-cosgrove-arm
Have make run serially for testing
a1c68c3 
Signed-off-by: Tom Cosgrove <tom.cosgrove@arm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@daverodgman daverodgman Awaiting requested review from daverodgman

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
component-test Test framework and CI scripts DO-NOT-MERGE priority-low Low priority - this may not receive review soon size-s Estimated task size: small (~2d)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tom-cosgrove-arm