Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@peterj
Copy link
Collaborator

@peterj peterj commented Oct 29, 2025

No description provided.

Copilot AI review requested due to automatic review settings October 29, 2025 13:13
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for specifying target platforms for Docker builds, enabling cross-platform builds (e.g., building linux/amd64 images on arm64 machines or vice versa).

Key changes:

  • Added Platform field to BuildCfg and DeployCfg structs to specify target Docker build platforms
  • Updated Docker build calls to include --platform flag when a platform is specified
  • Added --platform command-line flags to both build and deploy commands

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
go/cli/internal/cli/agent/deploy.go Added Platform field to DeployCfg struct with documentation and passed it to BuildCfg during deployment
go/cli/internal/cli/agent/build.go Added Platform field to BuildCfg struct and conditional logic to include --platform flag in docker build commands for both main agent and MCP server images
go/cli/cmd/kagent/main.go Added --platform flag definitions for both build and deploy commands

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@peterj peterj merged commit b777f44 into main Oct 29, 2025
22 checks passed
@peterj peterj deleted the peterj/addplatform branch October 29, 2025 13:22
killjoycircuit pushed a commit to killjoycircuit/kagent that referenced this pull request Nov 1, 2025
Signed-off-by: Peter Jausovec <[email protected]>
Signed-off-by: killjoycircuit <[email protected]>
artberger added a commit to kagent-dev/website that referenced this pull request Nov 20, 2025
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.

3 participants