Skip to content

Move test-arm-backend-vkml to OSDC - #23271

Merged
huydhn merged 1 commit into
mainfrom
osdc/arm-vkml
Sep 30, 2026
Merged

huydhn merged 1 commit into
mainfrom
osdc/arm-vkml

Conversation

@huydhn

@huydhn huydhn commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Last runner: linux.* and last linux_job_v2 caller in pull.yml. The job was added after the file was migrated, so it came in against the old template.

  • v2 → v3, needs: docker-image, hash-pinned image tag instead of the floating ci-image:<name> that only v2 resolves. Matches its sibling test-arm-backend-no-driver. Keeps the 24.04 arm-sdk image, which docker-builds.yml builds.
  • linux.2xlarge.memory is an r5.2xlarge (8 vCPU / 64 GiB), so mt-l-x86iavx512-8-64 is an exact match.
  • Drops sudo sysctl fs.inotify.max_user_watches=1048576. Nothing in this job watches files — the three legs run pytest, a cmake/ctest build, and a wrapper that scans a tee'd log — and there is no inotify reference anywhere in backends/arm or .ci. It arrived with the job in Arm backend: Enable Vulkan validation layer #22841, which does not mention it in the description or review. It could not have taken effect here regardless: it is a node-level knob and these pods are not privileged.

Testing

pull_request runs all three matrix legs (test_pytest_ops_vkml, test_ootb_tests_vgf, test_runtime_vgf) here.

@pytorch-bot

pytorch-bot Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/23271

Note: Links to docs will display an error until the docs builds have been completed.

✅ You can merge normally! (1 Unrelated Failure)

As of commit 7530fa4 with merge base fb60708 (image):

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 30, 2026
@github-actions

Copy link
Copy Markdown

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

This job was added after pull.yml was migrated, so it came in against the old
template: it was the last `runner: linux.*` and the last `linux_job_v2` caller
left in the file.

Matches its sibling test-arm-backend-no-driver: v3, `needs: docker-image`, and
the hash-pinned image tag rather than the floating `ci-image:<name>` that only
v2 resolves. Keeps the 24.04 arm-sdk image, which docker-builds.yml does build.

linux.2xlarge.memory is an r5.2xlarge, 8 vCPU / 64 GiB, so
mt-l-x86iavx512-8-64 is an exact match rather than a judgement call.

Also drops `sudo sysctl fs.inotify.max_user_watches`. Nothing in this job
watches files: the three legs run pytest, a cmake/ctest build, and a wrapper
that scans a `tee`'d log, and there is no inotify reference anywhere in
backends/arm or .ci. It arrived with the job in #22841, which does not mention
it in the description or review. It could not have taken effect here in any
case -- it is a node-level knob and these pods are not privileged -- so leaving
it would only have meant a guaranteed-failing command or a `|| true` implying
a dependency that does not exist.

Authored with Claude Code.
Comment thread .github/workflows/pull.yml
@huydhn
huydhn merged commit 476262f into main Sep 30, 2026
348 of 350 checks passed
@huydhn
huydhn deleted the osdc/arm-vkml branch September 30, 2026 19:00

This branch was successfully deployed

1 active deployment
cadence — 7530fa44 Deployed Sep 30, 2026 by huydhn via hifi-op-test / hifi4 #30557
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. topic: not user facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants