Skip to content

Conversation

@george-dorin
Copy link
Collaborator

@george-dorin george-dorin commented Sep 1, 2025


Below is a summarization created by an LLM (gpt-4-0125-preview). Be mindful of hallucinations and verify accuracy.

Why

The changes introduce enhancements to the observability framework, focusing on metrics collection and service discovery. The addition of process-exporter configuration and script for dynamic service discovery (sd-bridge.sh) aims to improve monitoring of containerized applications, enabling better performance insights and container management.

What

  • framework/observability/compose/conf/process-exporter.yaml
    • Added a new configuration file for process-exporter to define how processes are named and matched.
  • framework/observability/compose/conf/prometheus.yml
    • Introduced metric_relabel_configs for the cadvisor job to extract container IDs from various runtime formats.
    • Added a new job process_exporter with metric_relabel_configs to extract container IDs from the group name and adjust labels for better container identification.
    • Included a container-sd job with file_sd_configs, enhancing service discovery through file-based configurations.
  • framework/observability/compose/docker-compose.yaml
    • Added process-exporter service configuration, specifying image, container name, volumes, and commands for process metrics collection.
    • Introduced sd-bridge service for dynamic service discovery, including image, command, volumes, and environment variables configuration.
    • Included sd-targets volume to facilitate dynamic service discovery.
  • framework/observability/compose/scripts/sd-bridge.sh
    • Added a new script sd-bridge.sh for dynamic service discovery, capable of discovering Docker containers by label, pulling discovery JSON, merging, de-duplicating, and writing Prometheus file_sd JSON.

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.

1 participant