Skip to content

build: Remove redundant Go module downloads - #3346

Merged
EItanya merged 1 commit into
kagent-dev:mainfrom
timflannagan:cleanup/remove-go-mod-download
Oct 26, 2026
Merged

EItanya merged 1 commit into
kagent-dev:mainfrom
timflannagan:cleanup/remove-go-mod-download

Conversation

@timflannagan

Copy link
Copy Markdown
Contributor

Description

Remove the explicit go mod download steps from the shared Go, Codex harness,
and Claude harness Docker builds. The go build command automatically downloads
the modules required by each target, and every build step already mounts the same
BuildKit module and compilation caches.

The separate download vertices therefore do not improve cache reuse. They can
instead fetch dependencies that the selected image target does not need. All three
builder stages continue to resolve their dependencies and build successfully.

The Go command automatically downloads required modules during ordinary builds.
Each build step already mounts the persistent module and compilation caches, so
a separate download step does not improve cache reuse.

Removing the eager downloads also avoids fetching dependencies that the selected
image target does not need.

Signed-off-by: timflannagan <timflannagan@gmail.com>
@EItanya
EItanya added this pull request to the merge queue Oct 26, 2026
Merged via the queue into kagent-dev:main with commit c38912b Oct 26, 2026
31 checks passed
@timflannagan
timflannagan deleted the cleanup/remove-go-mod-download branch October 26, 2026 14:01
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