6 Commits (f8abbfd7d4ab7d9cd883fab5d24c8175eaec56b4)

Author	SHA1	Message	Date
Sebastien Tardif	922558fc1a	fix: address review feedback ... - Close log stream after reading (prevents connection/fd leak) - Strengthen tests to assert on output headers rather than error paths - Remove unused import Signed-off-by: Sebastien Tardif <SebTardif@ncf.ca>	2 months ago
Sebastien Tardif	854f7f6b72	fix: fetch logs from all containers in test pods ... When a test pod contains multiple containers (e.g. Istio/Consul/Vault sidecars), 'helm test --logs' failed with 'a container name must be specified'. This happened because GetPodLogs called the Kubernetes log API without specifying a container name. The fix fetches the pod spec first, then iterates over all containers (init containers + regular containers) and requests logs for each one explicitly. Errors from individual containers are collected and returned together via errors.Join rather than aborting on the first failure. Also fixes a typo: hooksByWight -> hooksByWeight. Closes #6902 Signed-off-by: Sebastien Tardif <SebTardif@ncf.ca>	2 months ago
Matheus Pimenta	59ece92bed	pkg/kube: introduce support for custom kstatus readers ... Signed-off-by: Matheus Pimenta <matheuscscp@gmail.com>	5 months ago
Evans Mungai	e751a70e84	fix(release): fix test compilation error ... Signed-off-by: Evans Mungai <mbuevans@gmail.com>	5 months ago
Stephanie Hohenberg	8cf4ad73c3	updates tests after rebase from master ... Signed-off-by: Stephanie Hohenberg <stephanie.hohenberg@gmail.com>	7 months ago
Stephanie Hohenberg	31131cff9b	Add tests to action package to improve coverage ... Signed-off-by: Stephanie Hohenberg <stephanie.hohenberg@gmail.com>	7 months ago