Skip to content

manifests: add metrics pipeline to the kind otel-collector#34

Open
Davanum Srinivas (dims) wants to merge 1 commit into
agent-substrate:mainfrom
dims:fix/kind-otel-metrics-pipeline
Open

manifests: add metrics pipeline to the kind otel-collector#34
Davanum Srinivas (dims) wants to merge 1 commit into
agent-substrate:mainfrom
dims:fix/kind-otel-metrics-pipeline

Conversation

@dims
Copy link
Copy Markdown
Contributor

@dims Davanum Srinivas (dims) commented May 21, 2026

atelet and ate-api-server push traces and metrics via OTLP. The kind collector declared only a traces: pipeline, so each metric push returned Unimplemented; the GKE managed collector accepts both.

Add a matching metrics: pipeline, re-exported via a Prometheus endpoint on :8889. Server-side configuration is unchanged.

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

  • Tests pass
  • Appropriate changes to documentation are included in the PR

atelet and ate-api-server push traces and metrics via OTLP. The kind
collector declared only a `traces:` pipeline, so each metric push
returned `Unimplemented`; the GKE managed collector accepts both.

Add a matching `metrics:` pipeline, re-exported via a Prometheus
endpoint on `:8889`. Server-side configuration is unchanged.

Signed-off-by: Davanum Srinivas <davanum@gmail.com>
@dims Davanum Srinivas (dims) changed the title manifests: add metrics pipeline to the kind otel-collector [WIP] manifests: add metrics pipeline to the kind otel-collector May 21, 2026
@dims Davanum Srinivas (dims) changed the title [WIP] manifests: add metrics pipeline to the kind otel-collector manifests: add metrics pipeline to the kind otel-collector May 21, 2026
prometheus:
endpoint: 0.0.0.0:8889
debug:
verbosity: detailed
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Not strictly related to your change, but we should probably switch to "normal" to not spam the logs.

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.

2 participants