Skip to content

Hosted-agents: first-cut agentName changes#835

Open
shawnyang-coder wants to merge 2 commits intomainfrom
shyang-agent-name
Open

Hosted-agents: first-cut agentName changes#835
shawnyang-coder wants to merge 2 commits intomainfrom
shyang-agent-name

Conversation

@shawnyang-coder
Copy link
Copy Markdown

@shawnyang-coder shawnyang-coder commented Apr 27, 2026

  • Add -name xxx flag to lk agent init/create/register/update, with create/deploy being required.
  • In init, agentName can optionally read from toml file, or the first command arg (even without --name flag).
  • Upon receiving responses, write agentName back to the toml file.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 27, 2026

CLA assistant check
All committers have signed the CLA.

Comment thread cmd/lk/agent.go
Action: createAgent,
Before: createAgentClient,
Flags: []cli.Flag{
agentNameFlag(true),
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

For Create and Deploy, I made the flag required. The intention is to force the migration.

Comment thread go.mod
k8s.io/apimachinery v0.34.1
)

replace github.com/livekit/server-sdk-go/v2 => ../server-sdk-go
Copy link
Copy Markdown
Author

@shawnyang-coder shawnyang-coder Apr 27, 2026

Choose a reason for hiding this comment

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

I need this line to build locally since this change spans multiple repos. How do I merge the PRs so that the two repos do not break each other during the interim? @yoonsio @real-danm

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I think we will need to merge the server-sdk-go PR first. Cut a release. Then update the go.mod here to point to that release right? But what about those livekit-cli used in the field?

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.

Yes that is correct

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