-
Notifications
You must be signed in to change notification settings - Fork 608
Arm build and runner CI #1111
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
Arm build and runner CI #1111
Conversation
.github/workflows/pull.yml
Outdated
uses: pytorch/test-infra/.github/workflows/linux_job.yml@main | ||
with: | ||
runner: linux.2xlarge | ||
docker-image: executorch-ubuntu-22.04-clang12 |
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.
You would need to use the new ARM image here once it's ready
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.
Yes! Please let me know when it's ready. So far I use the main image just for debugging
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.
@huydhn ETA when it will be ready? @digantdesai if Arm really want to enable it asap, we can enable it by installing the their simulator and toolchains on the fly like this.
.github/workflows/trunk.yml
Outdated
uses: pytorch/test-infra/.github/workflows/linux_job.yml@main | ||
with: | ||
runner: linux.2xlarge | ||
docker-image: executorch-ubuntu-22.04-clang12 |
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.
Same comment about using the new ARM image
This reverts commit 181fa12.
.github/workflows/pull.yml
Outdated
@@ -110,3 +110,21 @@ jobs: | |||
uses: ./.github/workflows/_unittest.yml | |||
with: | |||
docker-image: executorch-ubuntu-22.04-clang12 | |||
|
|||
test-arm-backend-delegation: |
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.
Are we running this on PRs? I think we should run all vendor-specific jobs ins trunk but with the ability to run it on PRs with labeling.
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.
Yes. I will remove this part. It's testing only.
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.
Thanks Hansong and Huy for helping on this
.ci/docker/build.sh
Outdated
@@ -21,11 +21,15 @@ BUCK2_VERSION=$(cat ci_commit_pins/buck2.txt) | |||
|
|||
case "${IMAGE_NAME}" in | |||
executorch-ubuntu-22.04-clang12) | |||
ARM_SDK=yes |
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.
Will remove this line in the end
@@ -41,6 +45,9 @@ BUILD_DOCS=1 | |||
# Copy requirements-lintrunner.txt from root to here | |||
cp ../../requirements-lintrunner.txt ./ | |||
|
|||
# Copy arm setup script from root to here | |||
cp -r ../../examples/arm/ ./arm |
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.
BTW how do we trigger an image build if examples/arm changes?
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.
we only should need setup.sh
and patches FWIW. But your question is valid.
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 harder to copy the specific files than the entire dir :p
.github/workflows/trunk.yml
Outdated
uses: pytorch/test-infra/.github/workflows/linux_job.yml@main | ||
with: | ||
runner: linux.2xlarge | ||
docker-image: executorch-ubuntu-22.04-clang12-arm-sdk |
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.
The name here should be executorch-ubuntu-22.04-arm-sdk
I think. Once you set the correct name, the failed jobs here https://github.com/pytorch/executorch/actions/runs/6712861373/job/18243312911 should use the correct Docker image
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.
Oops
@kirklandsign has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@kirklandsign merged this pull request in 2535a25. |
Add the setup part to docker image to make the run faster.