From d7a7ec6e1d21d73d7463fbf45586464e298627e8 Mon Sep 17 00:00:00 2001 From: Guang Yang Date: Tue, 10 Sep 2024 18:51:57 -0700 Subject: [PATCH] Updated the workflow to upload models to S3 (#5232) Summary: The upload should not be all or nothing ([example flow](https://github.com/pytorch/executorch/actions/runs/10783442883)). It should upload exported models to S3 if there is at least one artifact. Pull Request resolved: https://github.com/pytorch/executorch/pull/5232 Test Plan: - Android: https://github.com/pytorch/executorch/actions/runs/10800212616 - iOS: https://github.com/pytorch/executorch/actions/runs/10799346884 Reviewed By: huydhn Differential Revision: D62459630 Pulled By: guangy10 fbshipit-source-id: cbf6c1c9e030089096d126b91ec10a936030e15b --- .github/workflows/android-perf.yml | 1 + .github/workflows/apple-perf.yml | 1 + 2 files changed, 2 insertions(+) diff --git a/.github/workflows/android-perf.yml b/.github/workflows/android-perf.yml index c98fa98bb2..ba58435c69 100644 --- a/.github/workflows/android-perf.yml +++ b/.github/workflows/android-perf.yml @@ -178,6 +178,7 @@ jobs: upload-models: needs: export-models runs-on: linux.2xlarge + if: always() # Continue this job regardless of previous job outcome steps: - name: Download the models from GitHub uses: actions/download-artifact@v3 diff --git a/.github/workflows/apple-perf.yml b/.github/workflows/apple-perf.yml index bb7fd7b976..cb1b2b6a1b 100644 --- a/.github/workflows/apple-perf.yml +++ b/.github/workflows/apple-perf.yml @@ -179,6 +179,7 @@ jobs: upload-models: needs: export-models runs-on: linux.2xlarge + if: always() # Continue this job regardless of previous job outcome steps: - name: Download the models from GitHub uses: actions/download-artifact@v3