Skip to content

feat(cli): add AgentTemplate apply command - #2643

Open
codyhartsook wants to merge 8 commits into
kagent-dev:mainfrom
codyhartsook:agenttemplate-cli-lifecycle
Open

feat(cli): add AgentTemplate apply command#2643
codyhartsook wants to merge 8 commits into
kagent-dev:mainfrom
codyhartsook:agenttemplate-cli-lifecycle

Conversation

@codyhartsook

@codyhartsook codyhartsook commented Aug 31, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • add client-side apply commands for AgentTemplate manifests
  • route lifecycle operations through the generated AgentTemplate gRPC client
  • add unit and E2E coverage for the full manifest lifecycle

Tests

  • make -C go lint
  • KIND_CLUSTER_NAME=kagent-cli-refactor KAGENT_E2E_GRPC_TARGET=127.0.0.1:18084 make -C go e2e

@codyhartsook
codyhartsook requested a review from a team as a code owner August 31, 2026 21:00
@github-actions github-actions Bot added the enhancement New feature or request label Aug 31, 2026
EItanya
EItanya reviewed Sep 1, 2026
View reviewed changes

@EItanya EItanya left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we move to apply style create/update for our k8s resources?

@codyhartsook

codyhartsook commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

Can we move to apply style create/update for our k8s resources?

yeah sounds good

@github-actions github-actions Bot added enhancement New feature or request and removed enhancement New feature or request labels Sep 2, 2026
@codyhartsook codyhartsook reopened this Sep 2, 2026
@codyhartsook
codyhartsook marked this pull request as draft September 2, 2026 19:10
@codyhartsook
feat(cli): add AgentTemplate lifecycle commands
18514f2 
Signed-off-by: Cody Hartsook <cody.hartsook@solo.io>
@codyhartsook
test(cli): cover AgentTemplate lifecycle
2db8ff8 
Signed-off-by: Cody Hartsook <cody.hartsook@solo.io>
@codyhartsook
fix(api): persist AgentTemplate labels on update
e2b8420 
Signed-off-by: Cody Hartsook <cody.hartsook@solo.io>
@codyhartsook
refactor(cli): manage AgentTemplates through apply
7aadb24 
Signed-off-by: Cody Hartsook <cody.hartsook@solo.io>
@codyhartsook
refactor(cli): limit AgentTemplate lifecycle to apply
0e21198 
Signed-off-by: Cody Hartsook <cody.hartsook@solo.io>
@codyhartsook
refactor(cli): use local lifecycle client name
a328ef2 
Signed-off-by: Cody Hartsook <cody.hartsook@solo.io>
@codyhartsook
fix(cli): align AgentTemplate test with corev1 LocalObjectReference
45ffb4e 
Signed-off-by: Cody Hartsook <cody.hartsook@solo.io>
@codyhartsook
codyhartsook force-pushed the agenttemplate-cli-lifecycle branch from 60774e1 to 45ffb4e Compare September 2, 2026 20:57
@codyhartsook
codyhartsook marked this pull request as ready for review September 2, 2026 23:49
@codyhartsook codyhartsook changed the title feat(cli): add AgentTemplate lifecycle commands feat(cli): add AgentTemplate apply commands Sep 2, 2026
@github-actions github-actions Bot added enhancement New feature or request and removed enhancement New feature or request labels Sep 2, 2026
@codyhartsook
codyhartsook requested a review from EItanya September 3, 2026 14:38
@codyhartsook codyhartsook changed the title feat(cli): add AgentTemplate apply commands feat(cli): add AgentTemplate apply command Sep 3, 2026
@github-actions github-actions Bot added enhancement New feature or request and removed enhancement New feature or request labels Sep 3, 2026
EItanya
EItanya reviewed Sep 4, 2026
View reviewed changes
Comment thread go/core/test/e2e/cli_catalog_lifecycle_test.go
}
}

func TestE2ECLIAgentTemplateLifecycle(t *testing.T) {

@EItanya EItanya Sep 4, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This seems overkill for e2e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@EItanya EItanya EItanya left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@codyhartsook @EItanya