Skip to content

[ML] Fix CI analytics steps failing with exit code 127#3000

Merged
edsavage merged 2 commits intomainfrom
fix/ci-python-image
Mar 17, 2026
Merged

[ML] Fix CI analytics steps failing with exit code 127#3000
edsavage merged 2 commits intomainfrom
fix/ci-python-image

Conversation

@edsavage
Copy link
Contributor

@edsavage edsavage commented Mar 16, 2026

Summary

  • The analyze_build_timings, ingest_build_timings, and check_build_regression CI steps were failing with exit code 127 because they used the python:3-slim Docker image, which lacks curl and git — both required by the Buildkite agent environment hook.
  • Switches all three steps from python:3-slim to python:3 (the full image that includes these utilities).

Observed in: https://buildkite.com/elastic/ml-cpp-snapshot-builds/builds/6022#019cf4a2-72b5-4971-8218-9c21fff8df81

Test plan

  • Verify the analyze_build_timings step passes on the next snapshot build
  • Verify the ingest_build_timings step passes
  • Verify the check_build_regression step passes (on PR builds)

Made with Cursor

The python:3-slim image lacks curl and git, which the Buildkite
agent environment hook requires. This caused the analyze_build_timings,
ingest_build_timings, and check_build_regression steps to fail with
exit code 127 ("command not found").

Made-with: Cursor
@prodsecmachine
Copy link

prodsecmachine commented Mar 16, 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.

edsavage added a commit that referenced this pull request Mar 16, 2026
…step

Same fix as #3000 — python:3-slim lacks curl and git needed by the
Buildkite agent environment hook.

Made-with: Cursor
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 84d2d91 into main Mar 17, 2026
16 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