Skip to content

always set the KAGENT_NAMESPACE; add the agent URL to the deployed page#316

Merged
peterj merged 2 commits intomainfrom
peterj/fixdeploypart1
Mar 12, 2026
Merged

always set the KAGENT_NAMESPACE; add the agent URL to the deployed page#316
peterj merged 2 commits intomainfrom
peterj/fixdeploypart1

Conversation

@peterj
Copy link
Contributor

@peterj peterj commented Mar 11, 2026

Description

We must always set the KAGENT_NAMESPACE, even when deploying to docker. This PR also adds the agent URL to the /deployed page in the UI

Change Type

/kind fix

Changelog

add agent URL to the UI and fix the local docker deployment
Screenshot 2026-03-11 at 3 11 37 PM

Signed-off-by: Peter Jausovec <peter.jausovec@solo.io>
Copilot AI review requested due to automatic review settings March 11, 2026 22:11
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 ensures local/docker agent deployments always have KAGENT_NAMESPACE set, and enhances the /deployed UI page to display (and copy) the local agent endpoint URL.

Changes:

  • Default KAGENT_NAMESPACE to "default" when not provided during agent resolution.
  • Add local agent endpoint URL display + copy-to-clipboard action on the /deployed page for local deployments.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
ui/app/deployed/page.tsx Adds computed local agent endpoint URL display and copy button in the deployed resources UI.
internal/registry/platforms/utils/deployment_adapter_utils.go Forces a namespace to be set for resolved agents, defaulting to "default" when missing.

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

Signed-off-by: Peter Jausovec <peter.jausovec@solo.io>
@agentregistry-dev agentregistry-dev deleted a comment from Copilot AI Mar 11, 2026
Copy link
Collaborator

@inFocus7 inFocus7 left a comment

Choose a reason for hiding this comment

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

lgtm.

validated ui change with

make run
go run ./cmd/cli/main.go agent init adk python agent316
go run ./cmd/cli/main.go agent build agent316 --push
go run ./cmd/cli/main.go agent publish agent316 --version 1.0.0
go run ./cmd/cli/main.go agent deploy agent316 --version 1.0.0 --env GOOGLE_API_KEY=test
open http://localhost:12121/deployed

@peterj peterj added this pull request to the merge queue Mar 12, 2026
Merged via the queue into main with commit c769350 Mar 12, 2026
7 checks passed
@peterj peterj deleted the peterj/fixdeploypart1 branch March 12, 2026 15:59
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