SLEAK-5896: stop implying Grafana alert setup in exhaustive testing guide#213
SLEAK-5896: stop implying Grafana alert setup in exhaustive testing guide#213agustinoli wants to merge 3 commits into
Conversation
Per Mati's review, the cost-vs-risk intro no longer says "configure alerts in Grafana" — that reads as a how-to we don't cover. It now points to reviewing latency/error-rate metrics that aren't included by default. Also fixed the Section 6 anchor link, which was already broken (missing "y alertas"/"and Alerts" suffix).
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (2)
WalkthroughSe actualizan dos apartados de la guía en inglés y español: la recomendación de costo vs. riesgo pasa a apoyarse en la observabilidad existente, y el checklist de Grafana reduce las métricas mínimas exigidas a CPU y memoria. ChangesActualización de texto en guía de testing
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Section 6 already says Grafana dashboards include latency and error
rate by default, so the previous phrasing ("metrics we don't include
by default") contradicted it. What's not on by default is alerting on
those metrics — reworded to say that instead.
|
Buen catch — la Sección 6 ya asume que las métricas de latencia/error rate vienen por defecto en los dashboards de Grafana, así que decir que "no las incluimos por defecto" contradecía eso. Reescribí la frase (EN+ES) para que el matiz sea sobre las alertas (esas sí no vienen por defecto), no sobre las métricas. Push en 918f283. |
| Cada sección de esta guía tiene un costo: tiempo de preparación, overhead de infraestructura y esfuerzo de desarrollo. Antes de ejecutar una sección, evaluá si el riesgo que cubre es real para tu aplicación y si la inversión es proporcional. | ||
|
|
||
| A veces el mismo riesgo se puede mitigar de forma más económica. En lugar de montar un stack completo de load testing, por ejemplo, podés configurar alertas de latencia y tasa de errores en Grafana que te avisen antes de que el rendimiento degradado llegue a tus usuarios. La observabilidad y las alertas se cubren en la [Sección 6 — Observabilidad](#sección-6-observabilidad). | ||
| A veces el mismo riesgo se puede mitigar de forma más económica. En lugar de montar un stack completo de load testing, por ejemplo, podés seguirle el pulso a las métricas de latencia y tasa de errores en Grafana — no configuramos alertas sobre ellas por defecto, así que queda en vos revisarlas de forma proactiva y detectar el rendimiento degradado antes de que llegue a tus usuarios. La observabilidad se cubre en la [Sección 6 — Observabilidad y alertas](#sección-6-observabilidad-y-alertas). |
Cost-vs-risk intro now says "any monitoring stack" instead of naming Grafana specifically. Trimmed the default metrics list in the Observability section to what dashboards reliably show out of the box.
|
Cerrado a favor de #216, que consolida los cambios de SLEAK-5896, SLEAK-5875 y el fix de anchor en un solo PR. |
Summary
environment-exhaustive-testing-guide(EN+ES) — Mati flagged that it implies a how-to we don't cover.#section-6-observabilitywhile the heading isSection 6: Observability and Alerts).Ref: SLEAK-5896, follow-up on #201
Test plan
make build— no broken anchors on this page (EN+ES)Summary by CodeRabbit