chore: remove report-generator from all environments #209

Merged

admin merged 1 commits from remove-report-generator into main 2026-02-21 09:44:50 +01:00

Conversation 1 Commits 1 Files Changed 18 -879

claude commented 2026-02-21 09:43:23 +01:00

Owner

Copy Link

Decommissioning report-generator (load testing application).

Removed

• 6 ArgoCD app definitions
• 9 infra manifests (networkpolicy, secrets, seed-jobs)
• 3 Helm values files

K8s resources already deleted via ArgoCD cascade delete.

Decommissioning report-generator (load testing application). ## Removed - 6 ArgoCD app definitions - 9 infra manifests (networkpolicy, secrets, seed-jobs) - 3 Helm values files K8s resources already deleted via ArgoCD cascade delete.

claude added 1 commit 2026-02-21 09:43:23 +01:00

chore: remove report-generator from all environments ... 9acb62e515

Report-generator was a load testing application. Decommissioning:
- Remove ArgoCD app definitions (6 apps)
- Remove infra manifests (networkpolicy, secrets, seed-jobs)
- Remove Helm values (dev/staging/prod)

K8s resources already deleted via ArgoCD cascade delete.

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

claude commented 2026-02-21 09:43:32 +01:00

Author

Owner

Copy Link

AI Code Review

Обзор PR: Удаление report-generator

Статус: ✅ APPROVED — это чистое удаление выведенного из эксплуатации приложения.

Проверка

• Безопасность: Secrets удаляются корректно (no hardcoded values в diff). NetworkPolicy удаляются без остатков.
• Ресурсы: Job имел правильные limits/requests перед удалением (100m/256Mi req, 1CPU/512Mi limit).
• Надежность: Seed Job был настроен корректно (ttlSecondsAfterFinished: 3600, backoffLimit: 3, idempotent logic с проверкой COUNT > 0).
• Best practices:
  • Образ: pinned tag 17.6.0-debian-12-r0 ✓
  • securityContext: runAsNonRoot, drop ALL, RuntimeDefault ✓
  • Метаки: app labels присутствуют ✓

Замечания

Нет критических проблем. Удаление полное и согласованное (6 ArgoCD apps + 9 manifests + 3 values файла).

Рекомендация: Убедиться, что ArgoCD синхронизировал cascade delete для всех ресурсов в K8s перед merge. Если ресурсы уже удалены через ArgoCD (как указано в PR), можно мержить.

---

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

## AI Code Review # Обзор PR: Удаление report-generator **Статус:** ✅ **APPROVED** — это чистое удаление выведенного из эксплуатации приложения. ## Проверка - **Безопасность:** Secrets удаляются корректно (no hardcoded values в diff). NetworkPolicy удаляются без остатков. - **Ресурсы:** Job имел правильные limits/requests перед удалением (100m/256Mi req, 1CPU/512Mi limit). - **Надежность:** Seed Job был настроен корректно (ttlSecondsAfterFinished: 3600, backoffLimit: 3, idempotent logic с проверкой `COUNT > 0`). - **Best practices:** - Образ: pinned tag `17.6.0-debian-12-r0` ✓ - securityContext: runAsNonRoot, drop ALL, RuntimeDefault ✓ - Метаки: app labels присутствуют ✓ ## Замечания Нет критических проблем. Удаление полное и согласованное (6 ArgoCD apps + 9 manifests + 3 values файла). **Рекомендация:** Убедиться, что ArgoCD синхронизировал cascade delete для всех ресурсов в K8s перед merge. Если ресурсы уже удалены через ArgoCD (как указано в PR), можно мержить. --- _Automated review by Claude Haiku | advisory, does not block merge_

admin merged commit bf1bec1fda into main 2026-02-21 09:44:50 +01:00

admin deleted branch remove-report-generator 2026-02-21 09:44:50 +01:00

admin referenced this issue from a commit 2026-02-21 09:44:51 +01:00

Merge pull request 'chore: remove report-generator from all environments' (#209) from remove-report-generator into main

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

admin claude

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: claude/k8s-apps#209

No description provided.