Skip to content

feat(adk): transition to ADK #649

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
EItanya wants to merge 61 commits into from
Closed

feat(adk): transition to ADK #649

EItanya wants to merge 61 commits into from

Conversation

@EItanya
Copy link
Contributor

@EItanya EItanya commented Jul 24, 2025 •
edited
Loading

This PR move the backend agent engine of kagent from autogen to ADK

TODO:

  • Finish implementing model providers
  • Decide what to do about Memory
  • Fix UI to use A2A directly

EItanya added 28 commits July 15, 2025 16:24
@EItanya
adk
3ce73ab 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
Merge branch 'main' of github.com:kagent-dev/kagent into eitanya/adk
bffecd0
@EItanya
Merge branch 'main' of github.com:kagent-dev/kagent into eitanya/adk
f07ff46
@EItanya
tons of adk related changes
702ef10 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
minor changes
1e99a5c 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
API changes
3261919
@EItanya
adk-task-service
b3fe901 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
Merge branch 'main' of github.com:kagent-dev/kagent into eitanya/adk-…
4ffd30c 
…task-service
@EItanya
also added session code
44adef8 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
merge with upcoming PR
064e307 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
unit tests are all passing
734a181 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
removed all autogen python code
88a0cb4 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
use storage version
6c83afc 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
merge with main
44204b5 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
oops
342fe98 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
getting much closer
992ab47 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
reverted to v1alpha1 toolserver
ea27556 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
revert to v1alpha1
a93c57a 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
hacky user management, but things are returning
684cc87 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
working aside from getting the initial message
e5fec53 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
ids are being passed correctly in Python
6042a48 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
fixed CLI, figured out the bug
046229a 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
Merge branch 'main' of github.com:kagent-dev/kagent into eitanya/adk
e41c0c7
@EItanya
working
ed070d0 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
tracing works:
c23b750 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
Merge branch 'main' of github.com:kagent-dev/kagent into eitanya/adk
cf2b713
@EItanya
streaming and task history works{
3afa37b 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
remove unecessary logs
b3acf39 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
Copilot AI review requested due to automatic review settings July 24, 2025 19:34
@EItanya EItanya requested a review from ilackarms as a code owner July 24, 2025 19:34
@peterj
Copy link
Collaborator

peterj commented Jul 28, 2025

I was trying to fetch all agents and I am getting this:

❯  curl localhost:8083/api/agents
{"error":"Agent not found: failed to get model: record not found"}

Let me know when this is ready, so I can do the UI portion.

EItanya added 4 commits July 28, 2025 14:04
@EItanya
/agents works now
e79f11c 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
helm-unit-tests are passing
bf144ee 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
move workspace into packages
a006c89 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
update tools server
d684c87 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
yuval-k
yuval-k reviewed Jul 28, 2025
View reviewed changes
go/controller/internal/controller/agent_controller.go
Named("autogenagent").
Owns(&appsv1.Deployment{}).
Owns(&corev1.ConfigMap{}).
Owns(&corev1.Service{}).
Copy link
Collaborator

@yuval-k yuval-k Jul 28, 2025

Choose a reason for hiding this comment

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

Suggested change
Owns(&corev1.Service{}).
Owns(&corev1.Service{}).
Owns(&corev1.ServiceAccount{}).

EItanya and others added 12 commits July 28, 2025 20:03
@EItanya
add ServiceAccount to owned
76e8409 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
oops
e7e60c2 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
Merge branch 'main' into eitanya/adk
62d8eac
@peterj
fix the name used to get the agent
0809742 
Signed-off-by: Peter Jausovec <peter.jausovec@solo.io>
@EItanya
temporarily using upstream mcp/grafana in kagent pod
11e6ba5 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
Merge branch 'main' of github.com:kagent-dev/kagent into eitanya/adk
45171aa
@EItanya
add python release workflow, add READY status to printer columns
5fbbef4 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@peterj
Merge branch 'eitanya/adk' of github.com:kagent-dev/kagent into eitan…
11e333a 
…ya/adk

* 'eitanya/adk' of github.com:kagent-dev/kagent:
  add python release workflow, add READY status to printer columns
  temporarily using upstream mcp/grafana in kagent pod
  fix for 657 (#658)
@EItanya
moved some Python files, fixed mcp hang, sped up Python cold start
86adae0 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
Merge branch 'main' of github.com:kagent-dev/kagent into eitanya/adk
4042101
@EItanya
use proper version in the Dockerfile for kagent python
f3a50ac 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
update goldens
3fabaa7 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya EItanya marked this pull request as ready for review July 30, 2025 15:12
EItanya and others added 9 commits July 30, 2025 15:27
@EItanya
fix cli lint
41e33e7 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@peterj
Merge branch 'eitanya/adk' of github.com:kagent-dev/kagent into eitan…
0e2f939 
…ya/adk

* 'eitanya/adk' of github.com:kagent-dev/kagent:
  fix cli lint
  update goldens
  use proper version in the Dockerfile for kagent python
  moved some Python files, fixed mcp hang, sped up Python cold start
  docs: fix documentation links in README (#660)
@peterj
update ui to use a2a, update the Tool type and remove unused types
26ef383 
Signed-off-by: Peter Jausovec <peter.jausovec@solo.io>
@peterj
fix route path and tool categories
d56fe78 
Signed-off-by: Peter Jausovec <peter.jausovec@solo.io>
@EItanya
hmm
e73d0a7 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
Merge branch 'main' of github.com:kagent-dev/kagent into eitanya/adk
621003d
@EItanya
merge with main
2236e9f 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
go unit tests
a0562cd 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
fix helm unit tests
ee278cf 
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
Copy link
Contributor Author

EItanya commented Jul 31, 2025

Closing in favor of #671 as getting DCO to pass on this PR will be nearly impossible at this point.

@EItanya EItanya closed this Jul 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yuval-k yuval-k yuval-k left review comments

Copilot code review Copilot Copilot left review comments

@ilackarms ilackarms Awaiting requested review from ilackarms ilackarms is a code owner

@peterj peterj Awaiting requested review from peterj peterj is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@EItanya @dimetron @peterj @yuval-k