-
Notifications
You must be signed in to change notification settings - Fork 683
Android upload artifact #5588
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
Android upload artifact #5588
Changes from 12 commits
6fdeb9c
ce0df6d
4d9676b
17ae807
85981fc
64b3f0f
71aa6cb
c569ea8
fbf66e3
e1167b0
354b20f
95328cd
46f0114
307a6e9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,8 +23,8 @@ concurrency: | |
cancel-in-progress: true | ||
|
||
jobs: | ||
build-llm-demo: | ||
name: build-llm-demo | ||
build-aar: | ||
name: build-aar | ||
uses: pytorch/test-infra/.github/workflows/linux_job.yml@main | ||
with: | ||
runner: linux.2xlarge | ||
|
@@ -46,9 +46,38 @@ jobs: | |
# Build LLM Demo for Android | ||
bash build/build_android_llm_demo.sh ${ARTIFACTS_DIR_NAME} | ||
|
||
shasum -a 256 "${ARTIFACTS_DIR_NAME}/llm_demo/executorch.aar" | ||
|
||
upload-release-aar: | ||
name: upload-release-aar | ||
needs: build-aar | ||
if: ${{ startsWith(github.ref, 'refs/tags/android/release') }} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Just a note that if you use workflow_dispatch, you could run this step when the workflow event_type is workflow_dispatch. Let me find a concrete example There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ah, it's There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thank you Huy! I am trying with #5606 for that path |
||
runs-on: ubuntu-22.04 | ||
timeout-minutes: 10 | ||
permissions: | ||
id-token: write | ||
contents: read | ||
steps: | ||
- name: configure aws credentials | ||
uses: aws-actions/[email protected] | ||
with: | ||
role-to-assume: arn:aws:iam::308535385114:role/gha_executorch_upload-frameworks-android | ||
aws-region: us-east-1 | ||
- name: Upload AAR to RC if ciflow/android/release tag is present | ||
shell: bash | ||
run: | | ||
wget https://gha-artifacts.s3.amazonaws.com/${{ github.repository }}/${{ github.run_id }}/artifacts/llm_demo/executorch.aar | ||
shasum -a 256 executorch.aar > executorch.aar.sha256sums | ||
|
||
pip install awscli==1.32.18 | ||
AWS_CMD="aws s3 cp" | ||
VERSION_NAME="$(date +'%Y%m%d')" | ||
${AWS_CMD} executorch.aar s3://ossci-android/executorch/release/${VERSION_NAME}/executorch.aar --acl public-read | ||
${AWS_CMD} executorch.aar.sha256sums s3://ossci-android/executorch/release/${VERSION_NAME}/executorch.aar.sha256sums --acl public-read | ||
|
||
# Running Android emulator directly on the runner and not using Docker | ||
run-emulator: | ||
needs: build-llm-demo | ||
needs: build-aar | ||
# NB: Use metal install for KVM support to run the emulator faster | ||
runs-on: linux.24xl.spr-metal | ||
env: | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's better to refactor
build-arr
andupload-release-aar
to a separate dispatchable workflow that:However, that could come in a later PR if you prefer