-
Notifications
You must be signed in to change notification settings - Fork 429
Description
We currently have 3 workflows that run as part of the CI, and they run on both push and pull-request.
-
codeql.yml:
- Basic sanity check of the CodeQL behavior.
- Splits into 3 jobs of up to 9 minutes.
- Hopefully this will improve soon with Use smaller Plarform-Language packages when analyzing only one language #172 and similar changes
-
pr-checks.yml:
- Code quality checks
- Splits into 5 jobs of less than 30 seconds
-
integration-testing.yml:
- Extensive coverage of cross-feature behavior.
- Splits into 19 jobs of between 5 and 15 minutes.
- Use smaller Plarform-Language packages when analyzing only one language #172 is planning to add 21 additional jobs
All those parallel jobs compete for the same resources. Running them twice means that each push we are running 27x2 -> 54 jobs, filling up the queue.
Proposal
- Change
pr-checks.ymlto be only onpush(maybe rename tolinting?) - Change
codeql.ymlto be onpushonly formainandv1, andpull-requestfor all. - Change
integration-testingto be onpull-requestand only on push formainandv1 - Ideally sort the jobs in
integration-testingfrom the more likely to fail to the less likely to fail, and make the latter conditional on the former.- This is the strategy I am hoping to adopt for Use smaller Plarform-Language packages when analyzing only one language #172
This would avoid duplication of jobs without reducing the coverage we get.
Metadata
Metadata
Assignees
Labels
No labels