Skip to content

[ci]: skip unused Buildkite submodule checkout#1614

Merged
SolitaryThinker merged 1 commit into
mainfrom
maint/buildkite-skip-submodules
Jul 17, 2026
Merged

[ci]: skip unused Buildkite submodule checkout#1614
SolitaryThinker merged 1 commit into
mainfrom
maint/buildkite-skip-submodules

Conversation

@SolitaryThinker

Copy link
Copy Markdown
Collaborator

Summary

  • skip unused Cutlass, ThunderKittens, and VBench initialization on Buildkite agents before Modal launch
  • preserve clean checkouts and exact fork PR refs; Modal GPU jobs continue to initialize their own submodules before install, build, or test

Tests

  • bk pipeline validate --file .buildkite/pipeline.yml
  • pre-commit run --files .buildkite/pipeline.yml
  • Ruby YAML boolean assertion and git diff --check

No local pytest, GPU, dlcluster, or paid Modal job was run; this is a Buildkite checkout-policy-only change.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Buildkite pipeline configuration in .buildkite/pipeline.yml to disable automatic git submodule initialization by setting BUILDKITE_GIT_SUBMODULES: false, as remote jobs are responsible for initializing their own submodules. There are no review comments, and I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@mergify mergify Bot added type: ci CI/CD infrastructure scope: infra CI, tests, Docker, build labels Jul 17, 2026
@mergify

mergify Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🔴 1 of 1 protections blocking · waiting on 👀 reviews

Protection Waiting on
🔴 PR merge requirements 👀 reviews

🔴 PR merge requirements

Waiting for

  • #approved-reviews-by>=1
This rule is failing.
  • #approved-reviews-by>=1
  • check-success=fastcheck-passed
  • check-success=full-suite-passed
  • check-success~=pre-commit
  • title~=(?i)^\[(feat|feature|bugfix|fix|refactor|perf|ci|doc|docs|misc|chore|kernel|new.?model|skill|skills|infra)\]

@SolitaryThinker

Copy link
Copy Markdown
Collaborator Author

/test ssim

@SolitaryThinker
SolitaryThinker merged commit e2f8322 into main Jul 17, 2026
17 of 18 checks passed
@SolitaryThinker
SolitaryThinker deleted the maint/buildkite-skip-submodules branch July 17, 2026 01:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: infra CI, tests, Docker, build type: ci CI/CD infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant