Rework workflow

.github/actions/get-avd-info/action.yml

@@ -0,0 +1,19 @@
+name: "Get AVD Info"
+description: "Get the AVD info based on its API level."
+inputs:
+ api-level:
+ required: true
+outputs:
+ arch:
+ value: ${{ steps.get-avd-arch.outputs.arch }}
+ target:
+ value: ${{ steps.get-avd-target.outputs.target }}
+runs:
+ using: "composite"
+ steps:
+ - id: get-avd-arch
+ run: echo "arch=$(if [ ${{ inputs.api-level }} -ge 30 ]; then echo x86_64; else echo x86; fi)" >> $GITHUB_OUTPUT
+ shell: bash
+ - id: get-avd-target
+ run: echo "target=$(echo default)" >> $GITHUB_OUTPUT
+ shell: bash

.github/workflows/validate.yml

@@ -6,33 +6,27 @@ on:
  pull_request:
  types: [opened, synchronize]
 concurrency:
- group: ${{ github.workflow }}-${{ github.ref }}
+ # Limit concurrency to 1 for PRs. 'main' concurrency isn't limited.
+ group: ${{ github.head_ref || github.run_id }}
  cancel-in-progress: true
+env:
+ JAVA_VERSION: 11
 jobs:
  unit-tests:
  runs-on: ubuntu-latest
  timeout-minutes: 60
  strategy:
  fail-fast: false
  matrix:
- # Refactor to make these dynamic with a low/high bracket only on schedule, not push
- # For now this is just the fastest combo (api/arch/target/snapshot-warm-time) based on testing
- api-level: [29]
- arch: [x86_64]
- target: [google_apis]
- first-boot-delay: [600]
- # This is useful for benchmarking, do 0, 1, 2, etc (up to 256 max job-per-matrix limit) for averages
- iteration: [0]
- env:
- JAVA_TOOL_OPTIONS: -Xmx4g -XX:+UseParallelGC
+ api-level: [22, 26, 31]
  steps:
  - name: Checkout Repo
  uses: actions/checkout@v2
  - name: Setup Java
- uses: actions/setup-java@v2
+ uses: actions/setup-java@v4
  with:
  distribution: "adopt"
- java-version: "11"
+ java-version: ${{ env.JAVA_VERSION }}
  - name: Setup Gradle
  uses: gradle/gradle-build-action@v2
  timeout-minutes: 5
@@ -41,87 +35,24 @@ jobs:
  # Builds on other branches will only read from main branch cache writes
  # Comment this and the with: above out for performance testing on a branch
  cache-read-only: ${{ github.ref != 'refs/heads/main' }}
-
- - name: Warm Gradle Cache
- # This makes sure we fetch gradle network resources with a retry
- uses: nick-invision/retry@v2
+ # API 30+ emulators only have x86_64 system images.
+ - name: Get AVD info
+ uses: ./.github/actions/get-avd-info
+ id: avd-info
  with:
- timeout_minutes: 15
- retry_wait_seconds: 60
- max_attempts: 3
- command: ./gradlew :kotlin-audio:packageDebugAndroidTest --daemon
+ api-level: ${{ matrix.api-level }}
  - name: Enable KVM
  run: |
  echo 'KERNEL=="kvm", GROUP="kvm", MODE="0666", OPTIONS+="static_node=kvm"' | sudo tee /etc/udev/rules.d/99-kvm4all.rules
  sudo udevadm control --reload-rules
  sudo udevadm trigger --name-match=kvm
- # This appears to be 'Cache Size: ~1230 MB (1290026823 B)' based on watching action logs
- # Repo limit is 10GB; branch caches are independent; branches may read default branch cache.
- # We don't want branches to evict main branch snapshot, so save on main, read-only all else
- - name: AVD cache
- uses: actions/cache@v3
- id: avd-cache
- with:
- path: |
- ~/.android/avd/*
- ~/.android/adb*
- key: avd-${{ matrix.api-level }}-${{ matrix.arch }}-${{matrix.target}}-v1-${{ hashFiles('~/.android/avd/**/snapshots/**') }}
- restore-keys: |
- avd-${{ matrix.api-level }}-${{ matrix.arch }}-${{matrix.target}}-v1
- - name: AVD Boot and Snapshot Creation
- # Only generate a snapshot for saving if we are on main branch with a cache miss
- # Comment the if out to generate snapshots on branch for performance testing
- if: steps.avd-cache.outputs.cache-hit != 'true' && github.ref == 'refs/heads/main'
- uses: reactivecircus/android-emulator-runner@v2
- with:
- api-level: ${{ matrix.api-level }}
- force-avd-creation: false
- target: ${{ matrix.target }}
- arch: ${{ matrix.arch }}
- emulator-options: -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none
- sdcard-path-or-size: 100M
- disable-animations: true
- # Give the emulator a little time to run and do first boot stuff before taking snapshot
- script: echo "Generated AVD snapshot for caching."
-
- # This step is separate so pure install time may be calculated as a step
- - name: Emulator Snapshot After Firstboot Warmup
- # Only generate a snapshot for saving if we are on main branch with a cache miss
- # Switch the if statements via comment if generating snapshots for performance testing
- # if: matrix.first-boot-delay != '0'
- if: steps.avd-cache.outputs.cache-hit != 'true' && github.ref == 'refs/heads/main'
- env:
- FIRST_BOOT_DELAY: ${{ matrix.first-boot-delay }}
- uses: reactivecircus/android-emulator-runner@v2
- with:
- api-level: ${{ matrix.api-level }}
- force-avd-creation: false
- target: ${{ matrix.target }}
- arch: ${{ matrix.arch }}
- emulator-options: -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none
- sdcard-path-or-size: 100M
- disable-animations: true
- # Give the emulator a little time to run and do first boot stuff before taking snapshot
- script: |
- sleep $FIRST_BOOT_DELAY
- echo "First boot warmup completed."
  - name: Run Emulator Tests
  uses: reactivecircus/android-emulator-runner@v2
- timeout-minutes: 30
  with:
  api-level: ${{ matrix.api-level }}
- force-avd-creation: false
- target: ${{ matrix.target }}
- arch: ${{ matrix.arch }}
- emulator-options: -no-snapshot-save -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none
- sdcard-path-or-size: 100M
- disable-animations: true
- script: |
- $ANDROID_HOME/platform-tools/adb logcat '*:D' > adb-log.txt &
- sleep 5
- $ANDROID_HOME/platform-tools/adb shell su root "setprop ctl.restart zygote"
- sleep 10
- ./gradlew :kotlin-audio:connectedCheck --daemon
+ arch: ${{ steps.avd-info.outputs.arch }}
+ target: ${{ steps.avd-info.outputs.target }}
+ script: ./gradlew :kotlin-audio:connectedCheck
  - name: Publish Test Report
  uses: mikepenz/action-junit-report@v3
  if: always() # always run even if the previous step fails
@@ -132,10 +63,17 @@ jobs:
  steps:
  - name: Checkout Repo
  uses: actions/checkout@v2
- - name: Setup Java
- uses: actions/setup-java@v2
+ - uses: actions/setup-java@v4
  with:
  distribution: "adopt"
- java-version: "11"
+ java-version: ${{ env.JAVA_VERSION }}
+ - name: Setup Gradle
+ uses: gradle/gradle-build-action@v2
+ timeout-minutes: 5
+ with:
+ # Only write to the cache for builds on the 'main' branches, stops branches evicting main cache
+ # Builds on other branches will only read from main branch cache writes
+ # Comment this and the with: above out for performance testing on a branch
+ cache-read-only: ${{ github.ref != 'refs/heads/main' }}
  - name: Build Example App
  run: ./gradlew :kotlin-audio-example:assembleDebug