Skip to content

Export cache configuration as OTel metrics#50

Open
AlexanderKlump wants to merge 2 commits into
mainfrom
feature/config-metrics
Open

Export cache configuration as OTel metrics#50
AlexanderKlump wants to merge 2 commits into
mainfrom
feature/config-metrics

Conversation

@AlexanderKlump
Copy link
Copy Markdown
Contributor

Add config values to the s3_cache.info gauge (cache_enabled, listen_addr, upstream_region) and introduce separate config gauges for numeric values (cache_shards, cache_max_entries, cache_max_size_bytes, cache_max_object_size_bytes, cache_ttl_seconds, worker_threads).

This enables dashboards to compute cache utilization (e.g. size_bytes / config.cache_max_size_bytes) and to display the active configuration per instance.

Includes an integration test that verifies the full OTLP pipeline (proxy → OTel Collector → Prometheus exporter) and asserts correct Prometheus naming for all triggerable metrics.

@AlexanderKlump AlexanderKlump force-pushed the feature/config-metrics branch from f4a0fa3 to 654f193 Compare April 10, 2026 09:26
Add config values to the s3_cache.info gauge (cache_enabled, listen_addr,
upstream_region) and introduce separate config gauges for numeric values
(cache_shards, cache_max_entries, cache_max_size_bytes,
cache_max_object_size_bytes, cache_ttl_seconds, worker_threads).

This enables dashboards to compute cache utilization
(e.g. size_bytes / config.cache_max_size_bytes) and to display the
active configuration per instance.

Includes an integration test that verifies the full OTLP pipeline
(proxy → OTel Collector → Prometheus exporter) and asserts correct
Prometheus naming for all triggerable metrics.
@AlexanderKlump AlexanderKlump force-pushed the feature/config-metrics branch from 654f193 to 2153a06 Compare April 10, 2026 09:30
Comment thread tests/integration_metrics.rs Outdated
@AlexanderKlump AlexanderKlump marked this pull request as draft April 10, 2026 12:17
@AlexanderKlump AlexanderKlump marked this pull request as ready for review April 29, 2026 16:41
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