fix: increase resource quotas for VM migration #190

claude · 2026-02-20T11:04:19+01:00

claude commented

2026-02-20 11:04:19 +01:00

PG/MinIO removed from K8s namespaces. report-generator needs more CPU for in-Go aggregation. Prod quota supports HPA up to 5 replicas (5×4 CPU limits = 20).

claude added 1 commit 2026-02-20 11:04:20 +01:00

chore: increase resource quotas for VM migration

AI Review / AI Code Review (pull_request) Successful in 2s

Details

PR Checks / Validate & Security Scan (pull_request) Successful in 9s

Details

2881b388c4

PG/MinIO removed from K8s, report-generator needs more CPU for
in-Go aggregation. Prod quota supports HPA up to 5 replicas.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

admin approved these changes 2026-02-20 11:04:29 +01:00

claude commented

2026-02-20 11:04:34 +01:00

AI Code Review

Обзор PR: увеличение квот ресурсов

Общее впечатление: Логичное изменение, но требуются уточнения.

✅ Хорошо

Квоты растут пропорционально (2:1 для requests:limits)
pods: "30" и pods: "50" адекватны текущему числу приложений (23 в ArgoCD)
Prod квота 24 CPU поддерживает HPA до 5 реплик × 4 CPU как описано

⚠️ Критические вопросы

Обоснование prod: requests 8 → 8, limits 8 → 24
- Скачок limits на 16 CPU (2x) резко отличается от dev/staging (2x)
- Логика: report-generator требует больше CPU в prod, но мотивация не ясна
- Нужно: пересчёт требований report-generator или ссылка на CHANGELOG/ADR
Соответствие node capacity
- bare_srv_1: 16 CPU всего на 3 ноды = ~5.3 CPU на ноду
- Prod limits 24 CPU — пересвязка на физический лимит?
- Нужно: подтвердить, что узлы выдержат spike до 24 CPU (Calico, kernel overhead)
Отсутствие LimitRange
- ResourceQuota есть, но LimitRange (min/max per pod) не видна
- Риск: один pod может захватить весь лимит namespace'а
- Рекомендация: добавить LimitRange для dev/staging если нет
PDB не видна в diff
- Есть ли PodDisruptionBudget для prod (report-generator, critical apps)?
- С HPA до 5 реплик должны быть PDB minAvailable ≥ 2

🔍 Минорные

NetworkPolicy diff не показан — убедитесь, что не сломана сетевая изоляция
pods: "50" для prod — хватит ли при scale-up? (текущих 23 + резерв ~2x)

Вердикт: Одобрить при условии:

Добавить комментарий в коммите: обоснование 24 CPU limit для prod
Подтвердить node capacity + kernel overhead расчёты
Убедиться, что PDB покрывает критические workloads

Если это часть Phase 17 плана из ROADMAP, ссылка на ADR будет полезна.

Automated review by Claude Haiku | advisory, does not block merge

## AI Code Review ## Обзор PR: увеличение квот ресурсов **Общее впечатление:** Логичное изменение, но требуются уточнения. ### ✅ Хорошо - Квоты растут пропорционально (2:1 для requests:limits) - `pods: "30"` и `pods: "50"` адекватны текущему числу приложений (23 в ArgoCD) - Prod квота 24 CPU поддерживает HPA до 5 реплик × 4 CPU как описано ### ⚠️ Критические вопросы 1. **Обоснование prod: requests 8 → 8, limits 8 → 24** - Скачок limits на 16 CPU (2x) резко отличается от dev/staging (2x) - Логика: `report-generator` требует больше CPU в prod, но мотивация не ясна - **Нужно:** пересчёт требований report-generator или ссылка на CHANGELOG/ADR 2. **Соответствие node capacity** - bare_srv_1: 16 CPU всего на 3 ноды = ~5.3 CPU на ноду - Prod limits 24 CPU — пересвязка на физический лимит? - **Нужно:** подтвердить, что узлы выдержат spike до 24 CPU (Calico, kernel overhead) 3. **Отсутствие LimitRange** - ResourceQuota есть, но LimitRange (min/max per pod) не видна - Риск: один pod может захватить весь лимит namespace'а - **Рекомендация:** добавить LimitRange для dev/staging если нет 4. **PDB не видна в diff** - Есть ли PodDisruptionBudget для prod (report-generator, critical apps)? - С HPA до 5 реплик должны быть PDB minAvailable ≥ 2 ### 🔍 Минорные - NetworkPolicy diff не показан — убедитесь, что не сломана сетевая изоляция - `pods: "50"` для prod — хватит ли при scale-up? (текущих 23 + резерв ~2x) --- **Вердикт:** Одобрить **при условии**: 1. Добавить комментарий в коммите: обоснование 24 CPU limit для prod 2. Подтвердить node capacity + kernel overhead расчёты 3. Убедиться, что PDB покрывает критические workloads Если это часть Phase 17 плана из ROADMAP, ссылка на ADR будет полезна. --- _Automated review by Claude Haiku | advisory, does not block merge_

claude merged commit 965354ca3b into main

2026-02-20 11:05:25 +01:00

claude referenced this issue from a commit

2026-02-20 11:05:25 +01:00

Merge pull request 'fix: increase resource quotas for VM migration' (#190) from fix/resource-quotas into main

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: claude/k8s-apps#190