Skip to content

Comments

Instrument Docker Compose setups to enable Application Observability#228

Merged
ppcano merged 5 commits intomainfrom
pepe/fix-docker-compose-setups
Jul 28, 2025
Merged

Instrument Docker Compose setups to enable Application Observability#228
ppcano merged 5 commits intomainfrom
pepe/fix-docker-compose-setups

Conversation

@ppcano
Copy link
Collaborator

@ppcano ppcano commented Jul 25, 2025

  • Fixed the Tempo configuration so tracing works as expected locally.
  • Fixed the local Prometheus Data Source configuration to enable Drilldown app for Quickpizza metrics.
  • Made service.instance.id and service.namespace configurable for trace data.
  • Added service_name and service_namespace labels to metrics and logs in Docker Compose.
  • Enabled the host metric needed for Application Observability.

These changes should be safe for existing Kubernetes deployments — nothing should break there.

I'll open a new issue to discuss potential instrumentation enhancements.

Copy link
Contributor

@theSuess theSuess left a comment

Choose a reason for hiding this comment

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

Haven't run this locally but the changes look good to me.

We might want to look into using the faro receiver for the local setup to also have something resembling frontend-o11y in the local variant

@ppcano ppcano merged commit fcb2e65 into main Jul 28, 2025
13 checks passed
@ppcano ppcano deleted the pepe/fix-docker-compose-setups branch July 28, 2025 10:22
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