[Dependency Analysis] Add Android Gradle Plugin #21022
Merged
Conversation
This file contains 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
FYI: This commit adds a previous version of the plugin (1.28.0) and not the latest version (1.32.0). This is because the latest version gets stuck during the 'computeActualUsage' phase and increases the build time of the 'buildHealth' task by as much as five times (5x). For more info see: Stuck in computeActualUsageDebug for many minutes #1186 (https://github.com/autonomousapps/ dependency-analysis-gradle-plugin/issues/1186)
This commit updates the encrypted 'gradle.properties.enc' file to point to that pinned hash which has the 'dependency.analysis.android.ignored.variants' property added into the 'gradle.properties' file for the WPAndroid project. FYI: With this configuration only the 'debug' variant (build type) for the lib modules and 'wordpressJalapenoDebug' + 'jetpackJalapenoDebug' variants (product flavor + build type) for the app module is going to be included, with the rest of the variants ignored.
FYI: 1. This job will be then used by 'buildkite-ci' and configured as a 'buildkite_pipeline_schedule' with a weekly frequently. 2. This job will also notify on the android-core-notifs slack channel on a build failure. 3. This job has added custom meta-data for dependency analysis purposes. This is done in order for it to be queried and easily identifiable amongst other builds when working with the Buildkite's REST API.
|
|
 |
App Name |  Jetpack |
| Flavor | Jalapeno | |
| Build Type | Debug | |
| Version | pr21022-65e421a | |
| Commit | 65e421a | |
| Direct Download | jetpack-prototype-build-pr21022-65e421a.apk |
|
 |
App Name |  WordPress |
| Flavor | Jalapeno | |
| Build Type | Debug | |
| Version | pr21022-65e421a | |
| Commit | 65e421a | |
| Direct Download | wordpress-prototype-build-pr21022-65e421a.apk |
AliSoftware
approved these changes
Jun 28, 2024
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.
Project Thread: paaHJt-6yU-p2
Required By: BuildkiteCI#456
This PR adds the dependency analysis Android Gradle plugin to this project, for dependency analysis purposes. Also, a custom meta-data is added to the dependency analysis scheduled build so that it can be easily queried via Buildkite's REST API.
For now, only the main
buildHealthtask is going to be utilized and produce data once every week on CI (see this commit and this PR). Amongst other, this data will include the following:projectCount)unusedCount)undeclaredCount)misDeclaredCount)compileOnlyCount)runtimeOnlyCount)Afterward, this data will get collected from CI and uploaded to our Apps Metrics infrastructure, for visualization and alerting purposes.
To Test:
./gradlew buildHealthtask and verify that under the root levelbuild/reports/dependency-analysisfolder you get the below 2 reports both, in JSON and text format:build-health-report.jsonbuild-health-report.txtNew Build🟢 CI button for WPAndroid, test this standalone dependency analysis job (see form below). Then:/meta-datato that CI build's URL (example) will give you one extra meta-data, theschedule-buildone, with a value ofdependency-analysis(see screenshot below):build/reports/dependency-analysis/build-health-report.jsonbuild/reports/dependency-analysis/build-health-report.txthttps://api.buildkite.com/v2/organizations/automattic/pipelines/wordpress-android/builds?meta_data[scheduled-build]=dependency-analysisRegression Notes: (
N/A)PR Submission Checklist:
RELEASE-NOTES.txtif necessary.Testing Checklist (strike-out the not-applying and unnecessary ones):
N/A