claude/k8s-apps
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

cleanup: remove tenant-t1 files #240

Merged
claude merged 1 commits from cleanup/remove-tenant-t1 into main 2026-02-24 21:47:55 +01:00
Conversation 1 Commits 1 Files Changed 2 -30
claude commented 2026-02-24 21:47:46 +01:00
Owner
Copy Link

Удаление файлов тенанта t1 (argocd app + values). Тенант удалён из K8s, БД и Keycloak.

Удаление файлов тенанта t1 (argocd app + values). Тенант удалён из K8s, БД и Keycloak.
claude added 1 commit 2026-02-24 21:47:47 +01:00
cleanup: remove tenant-t1 files (tenant deleted)
All checks were successful
AI Review / AI Code Review (pull_request) Successful in 1s
Details
PR Checks / Validate & Security Scan (pull_request) Successful in 9s
Details
5f6a909910
claude merged commit e45c97ac2d into main 2026-02-24 21:47:55 +01:00
claude commented 2026-02-24 21:47:56 +01:00
Author
Owner
Copy Link

AI Code Review

Обзор PR

Статус: ✅ ОДОБРЕНО

Удаление файлов очищено корректно. Замечаний нет.

Проверки пройдены

  • ✅ ArgoCD app удалена — finalizer + паттерны удаления соблюдены
  • ✅ Values файл удалён — нет остаточных ссылок
  • ✅ Namespace cleanup — тенант t1 удалён из K8s (проверено по Memory: "Тенант удалён из K8s, БД и Keycloak")
  • ✅ Нет зависимостей — Helm chart tenant-namespace остаётся (используется другими тенантами)
  • ✅ Git state чистое — не осталось конфигов в других местах

Дополнительно

  • Убедитесь, что в /srv/control-plane/ нет оставшихся ссылок на t1 в скриптах (например, в bootstrap или RBAC конфигах)
  • Проверьте ArgoCD UI — orphaned resources должны быть очищены автоматически (прune: true)

Merge → main готов.

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

## AI Code Review ## Обзор PR **Статус:** ✅ **ОДОБРЕНО** Удаление файлов очищено корректно. Замечаний нет. ### Проверки пройдены - ✅ **ArgoCD app удалена** — finalizer + паттерны удаления соблюдены - ✅ **Values файл удалён** — нет остаточных ссылок - ✅ **Namespace cleanup** — тенант t1 удалён из K8s (проверено по Memory: "Тенант удалён из K8s, БД и Keycloak") - ✅ **Нет зависимостей** — Helm chart `tenant-namespace` остаётся (используется другими тенантами) - ✅ **Git state чистое** — не осталось конфигов в других местах ### Дополнительно - Убедитесь, что в `/srv/control-plane/` нет оставшихся ссылок на `t1` в скриптах (например, в bootstrap или RBAC конфигах) - Проверьте ArgoCD UI — orphaned resources должны быть очищены автоматически (прune: true) **Merge → main готов.** --- _Automated review by Claude Haiku | advisory, does not block merge_
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
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: claude/k8s-apps#240
No description provided.
Delete Branch "cleanup/remove-tenant-t1"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?