docs(python): correct KAgent controller port in package READMEs#1699
Merged
EItanya merged 1 commit intokagent-dev:mainfrom Apr 22, 2026
Merged
docs(python): correct KAgent controller port in package READMEs#1699EItanya merged 1 commit intokagent-dev:mainfrom
EItanya merged 1 commit intokagent-dev:mainfrom
Conversation
The quickstart examples for the kagent-openai and kagent-langgraph packages tell users to set kagent_url to http://localhost:8080. The kagent controller actually listens on 8083 (see helm/kagent/values.yaml controller.service.ports.port=8083, and the KAGENT_URL=8083 example in DEVELOPMENT.md). Port 8080 is the UI service. Copy-pasting the current examples against a port-forwarded controller results in connection refused errors. Updating the five occurrences across the two README files to match the actual controller port. Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Updates Python package READMEs to point quickstart examples at the correct local KAgent controller port (8083), aligning documentation with the Helm chart and existing development/test configuration.
Changes:
- Update
kagent_urlinkagent-openaiREADME quickstart and local development snippets fromlocalhost:8080tolocalhost:8083 - Update
httpx.AsyncClient(base_url=...)example inkagent-openaiREADME to uselocalhost:8083 - Update
kagent_urlinkagent-langgraphREADME quickstart fromlocalhost:8080tolocalhost:8083
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| python/packages/kagent-openai/README.md | Corrects controller connection URLs and the documented KAGENT_URL default to use port 8083 |
| python/packages/kagent-langgraph/README.md | Corrects the quickstart controller URL to use port 8083 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
EItanya
approved these changes
Apr 22, 2026
EItanya
added a commit
that referenced
this pull request
Apr 22, 2026
Same port issue as #1699 but in the sample READMEs. The crewai and langgraph samples tell users to set KAGENT_URL=http://localhost:8080 which is the UI port, not the controller. Updated all six occurrences across the three sample directories to 8083. --------- Signed-off-by: mesutoezdil <mesudozdil@gmail.com> Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io> Co-authored-by: Eitan Yarmush <eitan.yarmush@solo.io> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The quickstart examples in the kagent-openai and kagent-langgraph READMEs point to localhost:8080 but the controller runs on 8083. Port 8080 is the UI. Anyone copy-pasting those examples gets a connection refused straight away.
Changed all five occurrences across the two files to 8083.