promote: report-generator ea10e3b to prod #197

claude · 2026-02-20T16:20:22+01:00

claude commented

2026-02-20 16:20:22 +01:00

Automated promotion: report-generator image tag ea10e3b to prod

Automated promotion: report-generator image tag `ea10e3b` to prod

claude added 1 commit 2026-02-20 16:20:22 +01:00

promote: report-generator ea10e3b to prod

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

Details

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

Details

551711e8a9

claude commented

2026-02-20 16:20:33 +01:00

AI Code Review

Спасибо за PR. Пересмотрю manifest для report-generator в prod.

Проблемы:

Отсутствуют resource requests/limits — контейнер может потребить неограниченно CPU/память. В prod критично. Требуется добавить:
```
resources:
  requests:
    cpu: "100m"
    memory: "256Mi"
  limits:
    cpu: "500m"
    memory: "512Mi"
```
Нет health probes — Pod может быть в Running состоянии, но неработающий. Требуется livenessProbe и readinessProbe (HTTP, TCP или exec в зависимости от приложения).
Отсутствует PodDisruptionBudget — при планируемых maintenance K8s может одновременно evict все pods report-generator. Требуется PDB с minAvailable.
Отсутствует replica count — не вижу replicas в дифе. Если это deployment с replicas: 1, в prod недостаточно. Требуется минимум 2-3 для HA.
Отсутствуют labels/annotations — мало информации, но обычно требуется как минимум:
```
labels:
  app: report-generator
  version: ea10e3b
```
NetworkPolicy — убедитесь, что существует входящий NetworkPolicy для этого пода (в namespace prod). Если нет явного разрешения, трафик может быть заблокирован.
Image pull policy — проверьте, что imagePullPolicy: IfNotPresent или явно указан (не Always для внутреннего registry).

Что проверить перед merge:

Запущен ли pod успешно в staging с тагом ea10e3b?
Пройдены ли smoke-тесты?
Согласован ли tag с CHANGELOG?

Вердикт: ❌ Не одобряю до добавления resource requests/limits, health probes и PDB. Остальное критично для prod.

Покажите полный manifest (deployment/statefulset)?

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

## AI Code Review Спасибо за PR. Пересмотрю manifest для report-generator в prod. **Проблемы:** 1. **Отсутствуют resource requests/limits** — контейнер может потребить неограниченно CPU/память. В prod критично. Требуется добавить: ```yaml resources: requests: cpu: "100m" memory: "256Mi" limits: cpu: "500m" memory: "512Mi" ``` 2. **Нет health probes** — Pod может быть в Running состоянии, но неработающий. Требуется livenessProbe и readinessProbe (HTTP, TCP или exec в зависимости от приложения). 3. **Отсутствует PodDisruptionBudget** — при планируемых maintenance K8s может одновременно evict все pods report-generator. Требуется PDB с minAvailable. 4. **Отсутствует replica count** — не вижу replicas в дифе. Если это deployment с replicas: 1, в prod недостаточно. Требуется минимум 2-3 для HA. 5. **Отсутствуют labels/annotations** — мало информации, но обычно требуется как минимум: ```yaml labels: app: report-generator version: ea10e3b ``` 6. **NetworkPolicy** — убедитесь, что существует входящий NetworkPolicy для этого пода (в namespace `prod`). Если нет явного разрешения, трафик может быть заблокирован. 7. **Image pull policy** — проверьте, что `imagePullPolicy: IfNotPresent` или явно указан (не Always для внутреннего registry). **Что проверить перед merge:** - Запущен ли pod успешно в staging с тагом `ea10e3b`? - Пройдены ли smoke-тесты? - Согласован ли tag с CHANGELOG? **Вердикт:** ❌ **Не одобряю** до добавления resource requests/limits, health probes и PDB. Остальное критично для prod. Покажите полный manifest (deployment/statefulset)? --- _Automated review by Claude Haiku | advisory, does not block merge_

claude merged commit f3289324f9 into main

2026-02-20 16:20:38 +01:00

claude deleted branch promote/report-generator-ea10e3b-prod

2026-02-20 16:20:38 +01:00

claude referenced this issue from a commit

2026-02-20 16:20:39 +01:00

Merge pull request 'promote: report-generator ea10e3b to prod' (#197) from promote/report-generator-ea10e3b-prod into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: claude/k8s-apps#197