Skip to content

feat(deploy): add Helm teardown cleanup hooks#1663

Merged
msaad00 merged 3 commits intomainfrom
feat/helm-teardown-hooks
Apr 22, 2026
Merged

feat(deploy): add Helm teardown cleanup hooks#1663
msaad00 merged 3 commits intomainfrom
feat/helm-teardown-hooks

Conversation

@msaad00
Copy link
Copy Markdown
Owner

@msaad00 msaad00 commented Apr 22, 2026

Summary

  • add packaged Helm pre/post-delete cleanup hooks for product-owned in-cluster leftovers
  • label ExternalSecret-generated target secrets so the cleanup hook can remove them explicitly
  • document the updated destroy flow alongside the existing Terraform baseline and teardown helper

Testing

  • uv run pytest tests/test_deploy_k8s_cleanup.py tests/test_deploy_manifests.py -q
  • uv run ruff check src/agent_bom/deploy_k8s_cleanup.py tests/test_deploy_k8s_cleanup.py tests/test_deploy_manifests.py
  • uv run mypy src/agent_bom/deploy_k8s_cleanup.py
  • uv run mkdocs build --strict
  • PATH="/tmp/helm-bin/darwin-arm64:$PATH" python3 scripts/validate_helm_profiles.py --profile focused-pilot
  • PATH="/tmp/helm-bin/darwin-arm64:$PATH" python3 scripts/validate_helm_profiles.py --profile production

Closes #1635

@msaad00 msaad00 enabled auto-merge (squash) April 22, 2026 02:25
@msaad00 msaad00 merged commit eae9ca5 into main Apr 22, 2026
26 checks passed
@msaad00 msaad00 deleted the feat/helm-teardown-hooks branch April 22, 2026 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

P1: Helm uninstall cleanup path (hooks + Terraform + agent-bom teardown)

3 participants