Skip to content

Commit 9b0b8e7

Browse files
authored
Fix android perf periodic default spec
Differential Revision: D61346373 Pull Request resolved: #4736
1 parent 2b9c4b2 commit 9b0b8e7

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

.github/workflows/android-perf.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ on:
3434
description: The test spec to drive the test on AWS devices
3535
required: false
3636
type: string
37-
default: https://ossci-android.s3.amazonaws.com/executorch/android-llm-device-farm-test-spec.yml
3837
workflow_call:
3938
inputs:
4039
models:
@@ -65,7 +64,6 @@ on:
6564
description: The test spec to drive the test on AWS devices
6665
required: false
6766
type: string
68-
default: https://ossci-android.s3.amazonaws.com/executorch/android-llm-device-farm-test-spec.yml
6967

7068
concurrency:
7169
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref_name }}-${{ github.ref_type == 'branch' && github.sha }}-${{ github.event_name == 'workflow_dispatch' }}-${{ github.event_name == 'schedule' }}
@@ -268,6 +266,7 @@ jobs:
268266
# TODO: Hard code llm_demo_bpe for now in this job.
269267
android-app-archive: https://gha-artifacts.s3.amazonaws.com/${{ github.repository }}/${{ github.run_id }}/artifact/llm_demo_bpe/app-debug.apk
270268
android-test-archive: https://gha-artifacts.s3.amazonaws.com/${{ github.repository }}/${{ github.run_id }}/artifact/llm_demo_bpe/app-debug-androidTest.apk
271-
test-spec: ${{ inputs.test_spec }}
269+
# NB: Need to set the default spec here so that it works for periodic too
270+
test-spec: ${{ inputs.test_spec || 'https://ossci-android.s3.amazonaws.com/executorch/android-llm-device-farm-test-spec.yml' }}
272271
# Uploaded to S3 from the previous job
273272
extra-data: https://gha-artifacts.s3.amazonaws.com/${{ github.repository }}/${{ github.run_id }}/artifact/${{ matrix.model }}_${{ matrix.delegate }}/model.zip

0 commit comments

Comments
 (0)