Repository navigation
ci(android): [experiment, do not merge] AGP matrix UI tests on API 30 google_apis - #6495
Closed
sentry-junior[bot] wants to merge 1 commit into
Closed
sentry-junior[bot] wants to merge 1 commit into
sentry-junior[bot] wants to merge 1 commit into
Conversation
…e_apis Co-Authored-By: Roman Zavarnitsyn <roman.zavarnitsyn@sentry.io> #skip-changelog
📲 Install BuildsAndroid
|
sentry-junior Bot
added a commit
that referenced
this pull request
Nov 6, 2026
API 30 google_apis captures replay frames with PixelCopy (verified in #6495) and avoids the Android 15 ART method-tracing crash that the profiling tests hit on API 35. Co-Authored-By: Roman Zavarnitsyn <roman.zavarnitsyn@sentry.io>
An error occurred while trying to automatically change base from
jr/ci/agp-matrix-api35-replay
to
main
November 6, 2026 14:17
romtsn
added a commit
that referenced
this pull request
Nov 6, 2026
…Sauce Labs UI job (#6492) * ci(android): Run UI tests incl. replay on API 35 in AGP matrix, drop Sauce Labs UI job Co-Authored-By: Roman Zavarnitsyn <roman.zavarnitsyn@sentry.io> * ci(android): Pull replay snapshots even when the AGP matrix UI tests fail android-emulator-runner runs each script line in its own shell and stops at the first failing line, so a failing connectedReleaseAndroidTest skipped the snapshot pull. Run the tests and the pull from one script that exits with the tests' status. #skip-changelog * ci(android): Drop --daemon from the AGP matrix UI test run Co-Authored-By: Roman Zavarnitsyn <roman.zavarnitsyn@sentry.io> #skip-changelog * ci(android): Run the AGP matrix UI tests on API 30 google_apis API 30 google_apis captures replay frames with PixelCopy (verified in #6495) and avoids the Android 15 ART method-tracing crash that the profiling tests hit on API 35. Co-Authored-By: Roman Zavarnitsyn <roman.zavarnitsyn@sentry.io> * test(profiling): Run checkTimedOutProfile on emulators with a shorter timeout The emulator skip was added in #3074 because the timed-out profile's trace file was sometimes missing on emulators. Since #3478 timed-out profiles are discarded and the test asserts no profile is sent, so it no longer reads a trace file. Drop the skip, and add a test-only hook to shorten the 30s profiling timeout so the test waits 5s instead of 35s. Co-Authored-By: Roman Zavarnitsyn <roman.zavarnitsyn@sentry.io> * test(uitest): Remove checkTimedOutProfile The test only checks that a timed-out transaction profile is discarded, which AndroidProfilerTest and AndroidTransactionProfilerTest already cover. It was skipped on emulators, so nothing in CI ran it after the Sauce Labs job was removed. Also reverts the test-only profiling timeout hook. Co-Authored-By: Roman Zavarnitsyn <roman.zavarnitsyn@sentry.io> --------- Co-authored-by: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com> Co-authored-by: Roman Zavarnitsyn <roman.zavarnitsyn@sentry.io>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Experiment for #6492, do not merge.
This is stacked on #6492. It switches the AGP matrix emulator from API 35 to API 30, using the regular
google_apisimage instead of the API 30aosp_atdthatmainuses. The goal is to check whether PixelCopy, and with it the replay tests (ReplayTest,ReplaySnapshotTest,UserFeedbackUiTestwith replay on), works on API 30 when the image isn't ATD.On #6490, PixelCopy failed on API 30
aosp_atdwithERROR_SOURCE_NO_DATAand worked on API 35google_apis.Changes:
api-level: 35→30in both emulator steps, plus the matching AVD cache key.UPLOAD_REPLAY_SNAPSHOTS: false, so this branch doesn't upload snapshots to Sentry.Close this once the AGP matrix result is in.
#skip-changelog
via roman.
--
View Session [Sentry]