Skip to content

ci: use arm runners in integration tests#2877

Merged
tstirrat15 merged 1 commit intomainfrom
use-arm-in-integration-tests
Feb 4, 2026
Merged

ci: use arm runners in integration tests#2877
tstirrat15 merged 1 commit intomainfrom
use-arm-in-integration-tests

Conversation

@tstirrat15
Copy link
Contributor

@tstirrat15 tstirrat15 commented Feb 4, 2026

Description

In #2818 we started using ARM runners for most of our tests. This gave some huge speedups on unit tests; I punted on integration tests because the e2e tests were failing and I thought we'd need to make a bunch of updates.

It turns out that the e2e tests probably won't be able to run on ARM while they depend on Chaosd, but ARM runners sped up our integration tests by a factor of 2 (!!)

Changes

Use ARM runners for remaining tests where it helps

Testing

Review

@github-actions github-actions bot added area/api v1 Affects the v1 API area/dependencies Affects dependencies area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools) labels Feb 4, 2026
@codecov
Copy link

codecov bot commented Feb 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.79%. Comparing base (ea09c1e) to head (e4e2638).
⚠️ Report is 1 commits behind head on main.

❌ Your project status has failed because the head coverage (74.79%) is below the target coverage (75.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2877      +/-   ##
==========================================
+ Coverage   74.40%   74.79%   +0.40%     
==========================================
  Files         484      484              
  Lines       57786    57786              
==========================================
+ Hits        42989    43216     +227     
+ Misses      11787    11550     -237     
- Partials     3010     3020      +10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tstirrat15 tstirrat15 force-pushed the use-arm-in-integration-tests branch from 343dc28 to 8c539d1 Compare February 4, 2026 17:05
@github-actions github-actions bot removed area/api v1 Affects the v1 API area/dependencies Affects dependencies labels Feb 4, 2026
@tstirrat15 tstirrat15 force-pushed the use-arm-in-integration-tests branch from 8c539d1 to e4e2638 Compare February 4, 2026 17:08
@tstirrat15 tstirrat15 marked this pull request as ready for review February 4, 2026 17:30
@tstirrat15 tstirrat15 requested a review from a team as a code owner February 4, 2026 17:30
@miparnisari miparnisari changed the title Use arm runners in integration tests ci: use arm runners in integration tests Feb 4, 2026
@tstirrat15 tstirrat15 merged commit 3d98801 into main Feb 4, 2026
44 of 45 checks passed
@tstirrat15 tstirrat15 deleted the use-arm-in-integration-tests branch February 4, 2026 17:36
@github-actions github-actions bot locked and limited conversation to collaborators Feb 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants