Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Baseline Profiles] Test Generation (Without Login) #12429

Draft
wants to merge 10 commits into
base: trunk
Choose a base branch
from

Conversation

ParaskP7
Copy link
Contributor

TODO

@ParaskP7 ParaskP7 added status: do not merge Dependent on another PR, ready for review but not ready for merge. category: tooling Anything that involves building & maintaining the project, including scripts, `Fastfile`, etc. Core labels Aug 28, 2024
@ParaskP7 ParaskP7 changed the title [Baseline Profiles] Test Without Login [Baseline Profiles] Test Generation (Without Login) Aug 28, 2024
@ParaskP7 ParaskP7 force-pushed the test/baseline-profiles-without-login branch 6 times, most recently from f920f06 to 9e724d9 Compare August 28, 2024 11:23
For more info see:
- https://developer.android.com/topic/performance/baselineprofiles/
dex-layout-optimizations
- https://android-developers.googleblog.com/2023/05/
whats-new-in-jetpack-io-2023.html

"The plugin also allows you to easily automate the new Dex Layout
Optimization feature in AGP 8.1, which lets you define
BaselineProfileRule tests that collect classes used during startup,
and move them to the primary dex file in a multidex app to increase
locality. In a large app, this can improve cold startup time by 30% on
top of Baseline Profiles!"
For more info see: https://developer.android.com/topic/performance/
baselineprofiles/configure-baselineprofiles#store-into-sources
For more info see: https://developer.android.com/topic/performance/
baselineprofiles/configure-baselineprofiles#auto-generate-for-releases
For more info see: https://developer.android.com/topic/performance/
baselineprofiles/debug-baseline-profiles#installation_issues
FYI: This will very likely fail due to managed devices, but I want to
test it anyway so that I know how to further progress from here.
In order to use Gradle Managed Devices (GMD) on CI for a particular
device, it needs to be installed and its license accepted.

------------------------------------------------------------------------

FYI: After trying that, and without deeper investigation as to why, this
attempt fails on CI with the below:

[2024-08-28T08:39:58Z] [=== ] 10% Installing Google APIs Intel x86_6
Warning: Failed to read or create install properties file.
[2024-08-28T08:39:58Z] 🚨 Error: The command exited with status 1
[2024-08-28T08:39:58Z] user command error: exit status 1

It seems that the installation process does not have the necessary
permissions to write to the directory where the SDK is located, thus
failing.
…ssemble"

This reverts commit 44db206c5800ef845395ba52833318de84c8da8a.
AMI Name: android-build-image-6.12.0v1.2-rc-1

------------------------------------------------------------------------

FYI: This AMI includes the needed system image, 'system-images;
android-34;google_apis;x86_64', which is needed to be able for the
release build to use Gradle Managed Devices (GMD) with that particular
managed device configuration:

    testOptions.managedDevices.devices {
        pixel6Api34(ManagedVirtualDevice) {
            device = "Pixel 6"
            apiLevel = 34
            systemImageSource = "google"
        }
    }
@ParaskP7 ParaskP7 force-pushed the test/baseline-profiles-without-login branch from 9e724d9 to a3343f1 Compare September 13, 2024 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: tooling Anything that involves building & maintaining the project, including scripts, `Fastfile`, etc. Core status: do not merge Dependent on another PR, ready for review but not ready for merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant