Split ARM64 integration tests by area - #9172
Conversation
We _could_ tag just the base classes, but just tagging all the tests probably makes sense to me, as it's much more explicit, and less likely to get missed by agents/copy-pasting
|
@codex review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. What shall we delve into next? Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
Security review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
Execution-Time Benchmarks Report ⏱️Execution-time results for samples comparing This PR (9172) and master. ✅ No regressions detected |
Summary of changes
Split the arm64 integration tests by area
Reason for change
These are starting to take a long time, we should probably split them out into their target areas. Costs more in agent numbers, but reduces overall runtime as long as we don't saturate, and reduces chance of flake by running fewer unnecessary tests.
Implementation details
Just start doing the split
Test coverage
This is the test
Other details
Stacked on
[Trait("Area", "CiVisibility")]to CI Vis tests, and run in dedicated jobs only when required #9169