Skip to content

fix: Make submodule clone config an agent config#3752

Merged
DrJosh9000 merged 1 commit intomainfrom
a-987
Mar 12, 2026
Merged

fix: Make submodule clone config an agent config#3752
DrJosh9000 merged 1 commit intomainfrom
a-987

Conversation

@DrJosh9000
Copy link
Contributor

Description

BUILDKITE_GIT_SUBMODULE_CLONE_CONFIG was introduced in #1895 to fix a failing test, so it only made it as far as the job executor flags / env vars. Git config options are very powerful, so this should be controlled from the agent config.

Context

https://linear.app/buildkite/issue/A-987

Changes

  • Add the same flag and env var to agent_start.go
  • Propagate its value in job_runner.go:createEnvironment

Testing

  • Tests have run locally (with go test ./...). Buildkite employees may check this if the pipeline has run automatically.
  • Code is formatted (with go tool gofumpt -extra -w .)

Disclosures / Credits

I did not use AI tools at all

@DrJosh9000 DrJosh9000 requested review from a team as code owners March 10, 2026 06:05
Copy link
Contributor

@swebb swebb left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@DrJosh9000 DrJosh9000 merged commit ea7ff47 into main Mar 12, 2026
2 checks passed
@DrJosh9000 DrJosh9000 deleted the a-987 branch March 12, 2026 02:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants