Review: Grafana golden signals dashboard

review-1292-2026-06-01 Review

review ready

Verdict: READY

Template Completeness

  • [x] Type — Feature
  • [x] Lineage — standalone, observability audit
  • [x] Repo — ldraney/pal-e-platform
  • [x] User Story — clear
  • [x] Context — references existing dashboards
  • [ ] File Targets — mentions JSON but omits TF resource block (minor gap)
  • [x] Acceptance Criteria — dashboard visible in Grafana
  • [x] Test Expectations — visual verification
  • [x] Constraints — useful after metrics flowing
  • [x] Checklist — present
  • [x] Related — parent #43

Traceability

  • [x] story:observability — on board item
  • [x] arch:platform — on board item
  • [x] Forgejo issue #16 — open

File Targets

  • [x] terraform/dashboards/ — verified, existing app dashboards present as templates
  • [ ] terraform/modules/monitoring/main.tf — NOT in issue but required for kubernetes_config_map_v1 resource

Repo Placement

Correct. pal-e-platform owns Grafana dashboards.

Dependencies

  • [ ] #19 /metrics endpoint — PENDING
  • [ ] #15 ServiceMonitor — PENDING

Acceptance Criteria

Testable: dashboard appears in Grafana with request rate, error rate, latency, saturation panels.

Blast Radius

Low. New ConfigMap + dashboard only.

Decomposition Assessment

Two files (JSON + TF), one repo. Under 5 minutes.

Recommendation

  • Note in issue that main.tf resource block also needed.
  • Lower points from 3 to 2.