Skip to content

fix(query): wait briefly for dashboard fanout#17

Merged
smithclay merged 1 commit into
mainfrom
fix-grafana-query-admission
May 26, 2026
Merged

fix(query): wait briefly for dashboard fanout#17
smithclay merged 1 commit into
mainfrom
fix-grafana-query-admission

Conversation

@smithclay
Copy link
Copy Markdown
Owner

Summary

  • add a bounded query-admission wait for compatibility query routes so Grafana dashboard fanout can drain through short-lived heavy query slots
  • expose CANARDSTACK_QUERY_ADMISSION_WAIT_MS / _SECS and document the default
  • update the provisioned dashboard test to match the current OpenTelemetry demo dashboard

Validation

  • cargo test
  • cargo clippy --all-targets --all-features --locked -- -D warnings
  • rebuilt local compose image with docker compose -f compose.yaml -f compose.build.yaml up -d --build canardstack grafana
  • replayed six concurrent OTel demo Prometheus range queries: all returned 200
  • reloaded Grafana dashboard in browser: No data count 0

@smithclay smithclay marked this pull request as ready for review May 26, 2026 03:33
@smithclay smithclay merged commit 4639230 into main May 26, 2026
6 checks passed
@smithclay smithclay deleted the fix-grafana-query-admission branch May 26, 2026 04:59
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