fix: Recreate strategy for RWO PVC + cleanup GITEA_ADMIN_TOKEN #242

Recreate + RWO PVC — логично
- RWO (ReadWriteOnce) требует Recreate стратегию (RollingUpdate невозможна на разных узлах)
- Убедитесь, что replicaCount: 1 в values (иначе две реплики не смогут смонтировать один PVC одновременно)
Удаление GITEA_ADMIN_TOKEN — требует уточнения
- Переменная удалена без объяснения
- Проверьте:
  - Используется ли она в коде paas-portal? (grep -r GITEA_ADMIN_TOKEN)
  - Есть ли зависимость в OpenBao secret или env_file?
  - Это сделано намеренно (переход на claude-token) или случайно?
- Если переменная не нужна — хорошо, удалить; если нужна — PR будет нарушен
Отсутствие PR-описания
- Добавьте описание: почему Recreate, что с GITEA_ADMIN_TOKEN?

Проверьте дополнительно

Strategy timeout: Recreate может означать downtime. Если критично, добавьте progressDeadlineSeconds или убедитесь, что startup быстрый
PVC монтирование: Проверьте, что при переходе с RollingUpdate на Recreate старый pod удалится полностью (включая volume claims)
Секреты в OpenBao: Синхронизирована ли секретная пара (gitea-token и отсутствие gitea-admin-token) с /srv/control-plane/openbao/scripts/create-paas-secrets.sh?

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

Добавьте в commit message:

fix: Recreate strategy for RWO PVC + cleanup unused GITEA_ADMIN_TOKEN

- RWO PVC requires Recreate (RollingUpdate impossible across nodes)
- Verify replicaCount: 1 in values.yaml
- GITEA_ADMIN_TOKEN removed (unused; relying on GITEA_TOKEN)
- Sync OpenBao secret cleanup

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

## AI Code Review ## Обзор PR **Статус:** ✅ Одобрено с замечаниями ### Проблемы 1. **Recreate + RWO PVC** — логично - RWO (ReadWriteOnce) требует Recreate стратегию (RollingUpdate невозможна на разных узлах) - Убедитесь, что `replicaCount: 1` в values (иначе две реплики не смогут смонтировать один PVC одновременно) 2. **Удаление GITEA_ADMIN_TOKEN** — требует уточнения - Переменная удалена без объяснения - Проверьте: - Используется ли она в коде paas-portal? (grep -r GITEA_ADMIN_TOKEN) - Есть ли зависимость в OpenBao secret или env_file? - Это сделано намеренно (переход на claude-token) или случайно? - Если переменная не нужна — хорошо, удалить; если нужна — PR будет нарушен 3. **Отсутствие PR-описания** - Добавьте описание: почему Recreate, что с GITEA_ADMIN_TOKEN? ### Проверьте дополнительно - **Strategy timeout:** Recreate может означать downtime. Если критично, добавьте `progressDeadlineSeconds` или убедитесь, что startup быстрый - **PVC монтирование:** Проверьте, что при переходе с RollingUpdate на Recreate старый pod удалится полностью (включая volume claims) - **Секреты в OpenBao:** Синхронизирована ли секретная пара (`gitea-token` и отсутствие `gitea-admin-token`) с `/srv/control-plane/openbao/scripts/create-paas-secrets.sh`? ### Рекомендация Добавьте в commit message: ``` fix: Recreate strategy for RWO PVC + cleanup unused GITEA_ADMIN_TOKEN - RWO PVC requires Recreate (RollingUpdate impossible across nodes) - Verify replicaCount: 1 in values.yaml - GITEA_ADMIN_TOKEN removed (unused; relying on GITEA_TOKEN) - Sync OpenBao secret cleanup ``` --- _Automated review by Claude Haiku | advisory, does not block merge_

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: claude/k8s-apps#242