Skip to content

Stage Android emulator logcat output#3627

Draft
sjchmiela wants to merge 1 commit intomainfrom
stanley/eng-20762-stage-emulator-logcat
Draft

Stage Android emulator logcat output#3627
sjchmiela wants to merge 1 commit intomainfrom
stanley/eng-20762-stage-emulator-logcat

Conversation

@sjchmiela
Copy link
Copy Markdown
Contributor

Summary

  • stream adb logcat -v threadtime for every Android emulator started by eas/start_android_emulator
  • write live logcat output to a well-known staging directory under the build logs directory
  • record metadata for each stream so a later step can stop and collect logs safely

Why

When an Android emulator crashes before later diagnostics run, adb logcat -d may no longer work because the device is already gone. Streaming from startup preserves those logs without writing into directories that get archived.

Test plan

  • cd packages/build-tools && yarn test startAndroidEmulator AndroidEmulatorUtils

@linear
Copy link
Copy Markdown

linear Bot commented Apr 23, 2026

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 23, 2026

Codecov Report

❌ Patch coverage is 81.25000% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.50%. Comparing base (900ad76) to head (6c0f797).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...ages/build-tools/src/utils/AndroidEmulatorUtils.ts 78.58% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3627      +/-   ##
==========================================
- Coverage   55.70%   55.50%   -0.19%     
==========================================
  Files         852      856       +4     
  Lines       36549    36826     +277     
  Branches     7642     7646       +4     
==========================================
+ Hits        20356    20437      +81     
- Misses      14774    16294    +1520     
+ Partials     1419       95    -1324     

☔ 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.

@sjchmiela sjchmiela force-pushed the stanley/eng-20762-stage-emulator-logcat branch from 93e6a3d to 7945c7d Compare April 23, 2026 15:51
@sjchmiela sjchmiela force-pushed the stanley/eng-20762-stage-emulator-logcat branch from 7945c7d to 6c0f797 Compare April 23, 2026 15:56
@github-actions
Copy link
Copy Markdown

❌ It looks like a changelog entry is missing for this PR. Add it manually to CHANGELOG.md.
⏩ If this PR doesn't require a changelog entry, such as if it's an internal change that doesn't affect the user experience, you can add the "no changelog" label to the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant