refactor: remove MinIO from all environments #199

claude · 2026-02-20T17:59:26+01:00

claude commented

2026-02-20 17:59:26 +01:00

PDF stored in PostgreSQL BYTEA. MinIO no longer needed.

claude added 1 commit 2026-02-20 17:59:27 +01:00

refactor: remove MinIO from all environments

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

Details

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

Details

c25bc6c9ce

- Remove MINIO_* env vars from dev/staging/prod values
- Remove minio-access-key and minio-secret-key from secrets
- Remove port 9000 from NetworkPolicy egress rules
- PDF stored in PostgreSQL BYTEA, MinIO no longer needed

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

claude commented

2026-02-20 17:59:37 +01:00

AI Code Review

Обзор PR: Удаление MinIO

Вердикт: ✅ APPROVE — технически корректно.

Замечания

Секреты в plaintext: stringData в secrets.yaml видны в diff. Убедитесь, что эти файлы в .gitignore или используется sealed-secrets/OpenBao. Чувствительные данные не должны попадать в Git даже в удаляемых строках.
Миграция данных: убедитесь, что все PDF успешно перемещены в PostgreSQL BYTEA до развёртывания. После применения PR доступ к старым PDF из MinIO будет невозможен.
Очистка инфраструктуры: MinIO VM (185.47.204.228) и связанные ресурсы удалены из NetworkPolicy — проверьте, что сама VM/хранилище также удалены или отключены, чтобы избежать задержек на таймауты сети.
Тестирование: убедитесь, что report-generator в dev/staging корректно работает с PostgreSQL для хранения PDF (нет ошибок кодирования, лимитов BYTEA).

Как есть

NetworkPolicy синхронизированы во всех окружениях
Удаления последовательны (env vars, секреты, сетевые правила)
Нет осиротевших ссылок на MinIO в других приложениях

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

## AI Code Review ## Обзор PR: Удаление MinIO **Вердикт:** ✅ **APPROVE** — технически корректно. ### Замечания - **Секреты в plaintext:** `stringData` в `secrets.yaml` видны в diff. Убедитесь, что эти файлы в `.gitignore` или используется sealed-secrets/OpenBao. Чувствительные данные не должны попадать в Git даже в удаляемых строках. - **Миграция данных:** убедитесь, что все PDF успешно перемещены в PostgreSQL BYTEA до развёртывания. После применения PR доступ к старым PDF из MinIO будет невозможен. - **Очистка инфраструктуры:** MinIO VM (185.47.204.228) и связанные ресурсы удалены из NetworkPolicy — проверьте, что сама VM/хранилище также удалены или отключены, чтобы избежать задержек на таймауты сети. - **Тестирование:** убедитесь, что report-generator в dev/staging корректно работает с PostgreSQL для хранения PDF (нет ошибок кодирования, лимитов BYTEA). ### Как есть - NetworkPolicy синхронизированы во всех окружениях - Удаления последовательны (env vars, секреты, сетевые правила) - Нет осиротевших ссылок на MinIO в других приложениях --- _Automated review by Claude Haiku | advisory, does not block merge_

admin approved these changes 2026-02-20 17:59:51 +01:00

claude merged commit 3b5bf8e851 into main

2026-02-20 17:59:51 +01:00

claude referenced this issue from a commit

2026-02-20 17:59:53 +01:00

Merge pull request 'refactor: remove MinIO from all environments' (#199) from refactor/remove-minio 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#199