Skip to content

Add attributes to AgentConfigFile#329

Open
owais wants to merge 1 commit into
open-telemetry:mainfrom
owais:agent-file-attributes
Open

Add attributes to AgentConfigFile#329
owais wants to merge 1 commit into
open-telemetry:mainfrom
owais:agent-file-attributes

Conversation

@owais
Copy link
Copy Markdown

@owais owais commented May 7, 2026

Adds optional per-config-file key/value attributes so agents can report arbitrary metadata alongside each config body and content type.

Fixes #328

Adds optional per-config-file key/value attributes so agents can report arbitrary metadata alongside each config body and content type.

Fixes open-telemetry#328
@owais owais requested a review from a team as a code owner May 7, 2026 22:37
Copy link
Copy Markdown
Member

@tigrannajaryan tigrannajaryan left a comment

Choose a reason for hiding this comment

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

@owais and I discussed this in a call. Blocking for now while he is exploring an alternate option.

@RassK
Copy link
Copy Markdown
Contributor

RassK commented May 14, 2026

@owais and I discussed this in a call. Blocking for now while he is exploring an alternate option.

Any details why an alternative approach is needed? I was also thinking about the same approach. Vendor's spec can define the actual attributes while this spec is dynamic for everything.

@tigrannajaryan
Copy link
Copy Markdown
Member

Any details why an alternative approach is needed?

Alternative does not require any changes to opamp spec. It works with opamp as is, as it was intended.

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.

Add attributes to AgentConfigFile

3 participants