Skip to content

Conversation

@martinjt
Copy link
Member

@martinjt martinjt commented Dec 5, 2025

Changes

This adds a new service called telemetry-docs which serves a documentation site for the custom telemetry documented by the demo across all the services.

It also includes a weaver registry for the all those attributes.

This does not include a live-check interface, that will come later, nor does this include codegen for these attributes or metrics.

Merge Requirements

For new features contributions, please make sure you have completed the following
essential items:

  • CHANGELOG.md updated to document new feature additions
  • Appropriate documentation updates in the docs
  • Appropriate Helm chart updates in the helm-charts

Maintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.

@github-actions github-actions bot added the helm-update-required Requires an update to the Helm chart when released label Dec 5, 2025
@julianocosta89
Copy link
Member

@martinjt can we configure live-check to ensure the telemetry schema is always up to date?

I just don't want that to be another burden for maintainers.

Also, the k8s manifests shouldn't be part of this PR. That is generated via helm update whenever we have a new release

@martinjt
Copy link
Member Author

We can, i just didnt want to make it part of this PR since its already got a lot of moving parts.

I can remove the k8s bits.

@puckpuck puckpuck marked this pull request as ready for review December 17, 2025 14:12
@puckpuck puckpuck requested a review from a team as a code owner December 17, 2025 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

helm-update-required Requires an update to the Helm chart when released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants