Apply review comments

Author: addyess

src/kube_galaxy/pkg/components/_base.py

@@ -25,7 +25,7 @@
     format_component_pattern,
     install_binary,
 )
-from kube_galaxy.pkg.utils.errors import ComponentError
+from kube_galaxy.pkg.utils.errors import ClusterError, ComponentError
 from kube_galaxy.pkg.utils.logging import info
 from kube_galaxy.pkg.utils.shell import run
 
@@ -244,7 +244,10 @@ def bootstrap_hook(self) -> None:
                     raise ComponentError(
                         f"{comp_name} manifest not downloaded. Run download hook first."
                     )
-                apply_manifest(self.manifest_path)
+                try:
+                    apply_manifest(self.manifest_path)
+                except ClusterError as e:
+                    raise ComponentError(f"Failed to apply manifest for {comp_name}") from e
 
         pass
 

src/kube_galaxy/pkg/testing/spread.py

@@ -158,23 +158,6 @@ def _create_test_namespace(component_name: str) -> str:
         raise
 
 
-def _cleanup_test_namespace(namespace: str, timeout: int = 60) -> None:
-    """
-    Delete test namespace and wait for termination.
-
-    Args:
-        namespace: Namespace to delete
-        timeout: Maximum seconds to wait for deletion
-
-    Raises:
-        ClusterError: If namespace deletion fails
-    """
-    try:
-        delete_namespace(namespace, timeout)
-    except ClusterError:
-        raise
-
-
 def _generate_orchestration_spread_yaml(
     components: list[ComponentConfig], kubeconfig: Path
 ) -> list[str]:
@@ -390,7 +373,7 @@ def _run_component_tests(
             # Step 4: Cleanup namespace (always executed)
             if namespace:
                 try:
-                    _cleanup_test_namespace(namespace)
+                    delete_namespace(namespace)
                 except Exception as cleanup_exc:
                     warning(f"  Namespace cleanup failed: {cleanup_exc}")