Skip to content

chore(cloudagents): inject attributes header in byoc deploys#951

Merged
rektdeckard merged 3 commits into
mainfrom
tobiasfried/ap-956-byoc-deploy-attributes-are-not-persisted-on-agent-versions
Jul 15, 2026
Merged

chore(cloudagents): inject attributes header in byoc deploys#951
rektdeckard merged 3 commits into
mainfrom
tobiasfried/ap-956-byoc-deploy-attributes-are-not-persisted-on-agent-versions

Conversation

@rektdeckard

@rektdeckard rektdeckard commented Jul 15, 2026

Copy link
Copy Markdown
Member

https://linear.app/livekit/issue/AP-956/byoc-deploy-attributes-are-not-persisted-on-agent-versions

What: injects any user-defined deployment attributes as a request header (X-LIVEKIT-AGENT-ATTRIBUTES) after completing image upload in a BYOC deployment.

BYOC agent deployments are currently a 2-step process:

  1. SDK requests a push endpoint from cloud-agents service (generated by crane) to upload an image to
  2. SDK calls cloud-agents/v2 callback endpoint when the push is complete

The cloud-agents service then creates a record of the agent version and spins it up. In order to support custom user-defined attributes in BYOC cloud agent deployments, we have to forward them to cloud-agents svc after pushing the image, because this transaction is stateless from the perspective of cloud-agents svc.

In the Non-BYOC case, which is very similar, we had been appending these attributes as URL parameters. For consistency's sake, I have now standardized both cases to use the header value (with backward compatibility for the parameter).

REF: https://github.com/livekit/cloud-agents/pull/736, livekit/livekit-cli#914

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@rektdeckard
rektdeckard force-pushed the tobiasfried/ap-956-byoc-deploy-attributes-are-not-persisted-on-agent-versions branch from 953e9d5 to 5fedb49 Compare July 15, 2026 20:19
@yoonsio

yoonsio commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Can we do X-LIVEKIT-AGENT-VERSION-ATTRIBUTES instead? We are going to have support for other attributes in the future.

@rektdeckard

Copy link
Copy Markdown
Member Author

@yoonsio sure, makes sense

@rektdeckard
rektdeckard merged commit 0a3b2b9 into main Jul 15, 2026
7 checks passed
@rektdeckard
rektdeckard deleted the tobiasfried/ap-956-byoc-deploy-attributes-are-not-persisted-on-agent-versions branch July 15, 2026 20:47
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.

2 participants