Add nginx proxy timeout 300s for report-generator #205

claude · 2026-02-20T23:03:28+01:00

claude commented

2026-02-20 23:03:28 +01:00

Yearly reports take ~160s (bootstrap resampling). Default 60s nginx timeout causes 504.

claude added 1 commit 2026-02-20 23:03:29 +01:00

Add nginx proxy timeout 300s for report-generator

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

Details

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

Details

232d2bdd97

Yearly reports take ~160s (bootstrap resampling of 1.7M rows).
Default nginx timeout of 60s causes 504 Gateway Timeout.

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

claude commented

2026-02-20 23:03:40 +01:00

AI Code Review

Код review: Add nginx proxy timeout 300s for report-generator

✅ Финальный вердикт: Мерж одобрен.

Замечания (минорные)

Pod health probes: Убедитесь, что livenessProbe и readinessProbe на report-generator учитывают 160s bootstrap. Если probe timeout < 160s или failureThreshold отправит pod в Pending, увеличьте initialDelaySeconds и timeoutSeconds.
Backend readiness: Проверьте, что сам pod корректно отвечает на readiness запросы во время 160s обработки (не вешает соединение). Иначе ingress будет отправлять трафик на неготовый pod, и 300s timeout всё равно даст 504.

✓ Соответствие стандартам

Категория	Статус	Примечание
Security	✓	Annotation — не привилегированная операция, PSS compliant
Resource mgmt	✓	Timeout — не влияет на CPU/memory requests/limits
Reliability	⚠️	Timeout подходит для use case, но нужна проверка probe config
Best practices	✓	Одинаковые значения dev/staging/prod логичны (одна логика, разные домены)

Рекомендация

Перед мержем добавьте в PR description подтверждение:

Текущие probe settings (initialDelaySeconds, timeoutSeconds, failureThreshold)
Результат тестирования: report за 160s ≠ 504 после этого изменения

Это минимизирует risk регрессии.

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

## AI Code Review ## Код review: Add nginx proxy timeout 300s for report-generator ✅ **Финальный вердикт:** Мерж одобрен. ### Замечания (минорные) - **Pod health probes:** Убедитесь, что `livenessProbe` и `readinessProbe` на report-generator учитывают 160s bootstrap. Если probe timeout < 160s или failureThreshold отправит pod в Pending, увеличьте `initialDelaySeconds` и `timeoutSeconds`. - **Backend readiness:** Проверьте, что сам pod корректно отвечает на readiness запросы *во время* 160s обработки (не вешает соединение). Иначе ingress будет отправлять трафик на неготовый pod, и 300s timeout всё равно даст 504. ### ✓ Соответствие стандартам | Категория | Статус | Примечание | |-----------|--------|-----------| | **Security** | ✓ | Annotation — не привилегированная операция, PSS compliant | | **Resource mgmt** | ✓ | Timeout — не влияет на CPU/memory requests/limits | | **Reliability** | ⚠️ | Timeout подходит для use case, но нужна проверка probe config | | **Best practices** | ✓ | Одинаковые значения dev/staging/prod логичны (одна логика, разные домены) | ### Рекомендация Перед мержем добавьте в PR description подтверждение: - Текущие probe settings (*initialDelaySeconds, timeoutSeconds, failureThreshold*) - Результат тестирования: report за 160s ≠ 504 после этого изменения Это минимизирует risk регрессии. --- _Automated review by Claude Haiku | advisory, does not block merge_

admin approved these changes 2026-02-20 23:03:41 +01:00

admin left a comment

LGTM

claude merged commit 612dc1906e into main

2026-02-20 23:04:18 +01:00

claude referenced this issue from a commit

2026-02-20 23:04:18 +01:00

Merge pull request 'Add nginx proxy timeout 300s for report-generator' (#205) from add-ingress-timeout 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#205