Skip to content

[ML] Default to Linux x86_64 only for QA/PyTorch test comments#3001

Merged
edsavage merged 1 commit intoelastic:mainfrom
edsavage:fix/qa-test-platform-defaults
Mar 20, 2026
Merged

[ML] Default to Linux x86_64 only for QA/PyTorch test comments#3001
edsavage merged 1 commit intoelastic:mainfrom
edsavage:fix/qa-test-platform-defaults

Conversation

@edsavage
Copy link
Contributor

@edsavage edsavage commented Mar 19, 2026

Summary

  • When buildkite run_qa_tests or buildkite run_pytorch_tests is triggered via a PR comment without explicit on <platform> or <arch> options, default to building only Linux x86_64 instead of all platforms and architectures.
  • QA and PyTorch tests only run against Linux x86_64 builds (see pipeline.json.py lines 52-61), so the Windows, macOS, and aarch64 builds were wasted CI resources.
  • Explicit platform/arch overrides (e.g. buildkite run_qa_tests on linux, macos x86_64, aarch64) continue to work as before.

Test plan - after this is merged to main

  • Comment buildkite run_qa_tests on a PR and verify only the Linux x86_64 build + QA tests run
  • Comment buildkite run_qa_tests on linux, macos x86_64, aarch64 and verify platform/arch overrides still work
  • Comment buildkite build and verify all platforms still build as before

Made with Cursor

When "buildkite run_qa_tests" or "buildkite run_pytorch_tests" is
triggered via a PR comment without explicit platform/arch options,
default to building only Linux x86_64 instead of all platforms.
QA and PyTorch tests only run against Linux x86_64 builds so the
Windows, macOS, and aarch64 builds were unnecessary.

Made-with: Cursor
@prodsecmachine
Copy link

prodsecmachine commented Mar 19, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Copy link
Contributor

@valeriy42 valeriy42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@edsavage edsavage merged commit 47010b2 into elastic:main Mar 20, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants