fix(k8s): query resources by label in down() for proper cleanup #987

Merged

AFDudley merged 1 commits from fix-down-cleanup-by-label into main

2026-02-03 22:57:53 +00:00

Author	SHA1	Message	Date
A. F. Dudley	47d3d10ead	fix(k8s): query resources by label in down() for proper cleanup Some checks failed Lint Checks / Run linter (push) Failing after 14s Details Lint Checks / Run linter (pull_request) Failing after 15s Details K8s Deployment Control Test / Run deployment control suite on kind/k8s (pull_request) Failing after 2m3s Details Deploy Test / Run deploy test suite (pull_request) Successful in 2m10s Details K8s Deploy Test / Run deploy test suite on kind/k8s (pull_request) Failing after 2m51s Details Webapp Test / Run webapp test suite (pull_request) Successful in 4m0s Details Smoke Test / Run basic test suite (pull_request) Successful in 3m56s Details Previously, down() generated resource names from the deployment config and deleted those specific names. This failed to clean up orphaned resources when deployment IDs changed (e.g., after force_redeploy). Changes: - Add 'app' label to all resources: Ingress, Service, NodePort, ConfigMap, PV - Refactor down() to query K8s by label selector instead of generating names - This ensures all resources for a deployment are cleaned up, even if the deployment config has changed or been deleted Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>	2026-02-03 17:55:14 -05:00