Code Review дашбордов¶

Дашборды — такой же код. И точно так же нуждаются в ревью.

Только вообразите: разработчик добавил новую панель "CPU Usage per User" с группировкой by (user_id) по миллиону уникальных пользователей. На production. На что это похоже? На кнопку "Выключить всё". Code Review дашбордов предотвращает подобные катастрофы.

Что проверять¶

1. Избыточность панелей и запросов¶

Нет ли повторов? Не делает ли панель то же, что соседняя? Отсекайте дубликаты. Один дашборд — один полный обзор системы без повторов.

2. Эффективность запросов¶

Примените те же фильтры — см. query optimization. Отдельно отметьте: - Есть ли Recording Rules для тяжёлых вычислений - Используются ли макросы $__interval, $__range?

3. Правильность визуализации¶

Настроена ли легенда? Подписаны ли оси? Единицы измерения? Цвета? Не использует ли человек тип визуализации Table вместо Time Series для метрик?

4. Покрытие метрик (Coverage)¶

Проверьте: - SLO SLI покрыты - Критические ошибки покрыты - "Золотые сигналы" (latency, traffic, errors, saturation) — покрыты

5. Консистентность¶

Проверьте соответствие naming conventions: название дашборда, tags, description. Описание — написано ли оно?

PR шаблон проверки дашборда¶

## Dashboard Review Checklist

- [ ] Нет дублирующихся панелей
- [ ] Все запросы используют метки фильтрации и $__interval
- [ ] SLO покрыто, покрытие > 95% метрик производственного контура
- [ ] Названия тегов, описание добавлены
- [ ] Указан владелец (команда)
- [ ] Легенда читаема (ахроматические цвета или специальная цветовая схема)
- [ ] Тесты (графики данных с другими временными отрезками) — прошли

Использовать этот шаблон при каждом MR для дашбордов.

Далее: Чек-лист хорошего дашборда