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

Group small tests in CI #145

Open
daverodgman opened this issue Jan 2, 2024 · 4 comments
Open

Group small tests in CI #145

daverodgman opened this issue Jan 2, 2024 · 4 comments
Labels
enhancement New feature or request priority-high size-s Estimated task size: small (~2d)

Comments

@daverodgman
Copy link
Contributor

Group small (fast) test components into a small number of groups (maybe 1 group) to reduce CI overhead.

Ideal group size should probably be up to 20-30 mins - given that many small components are < 1 min, that should enable a lot of reduction of overhead.

Identification of components can be done manually (@daverodgman can supply recent data on component durations). Automated creation & updating of groups is way too low cost-benefit.

Ideally we probably want to retain the ability to treat these as individual components when not running in the CI.

Probably grouping can be implemented by naming components with the form component_xyz_group1, and then tweaking all.sh to behave accordingly when running in CI.

@daverodgman daverodgman added this to CI perf in EPICs for Mbed TLS Jan 2, 2024
@daverodgman daverodgman added enhancement New feature or request size-s Estimated task size: small (~2d) priority-high labels Jan 2, 2024
@gilles-peskine-arm
Copy link
Contributor

I've already done the all.sh side of the work. See #69.

Note that this comes at a price: the output is harder to analyze. So we'd trade off CI load for developer time, which might not be a good trade-off.

@gilles-peskine-arm gilles-peskine-arm closed this as not planned Won't fix, can't repro, duplicate, stale Jan 2, 2024
@daverodgman
Copy link
Contributor Author

@gilles-peskine-arm please don't close my issues

@daverodgman daverodgman reopened this Jan 2, 2024
@gilles-peskine-arm
Copy link
Contributor

Uh? Why do you want to keep a duplicate open? #69 has the same requirement and a more detailed analysis.

@daverodgman
Copy link
Contributor Author

I'm trying to organise an epic for some things that we may wish to look at as part of the Q1 tech debt work. Having my issues mysteriously vanish during this process, without even a heads-up, is incredibly unhelpful.

Within the Q1 tech debt sub-team, we will review these, the trade-offs, and ultimately make the decisions about which ones we are going to work on.

@minosgalanakis minosgalanakis removed this from CI perf in EPICs for Mbed TLS May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority-high size-s Estimated task size: small (~2d)
Projects
Development

No branches or pull requests

2 participants