From fd49a49b21989973c482677e5ed7fd28a7d42ba9 Mon Sep 17 00:00:00 2001 From: kryanbeane Date: Tue, 10 Jun 2025 15:08:19 +0100 Subject: [PATCH] feat: add comment about tearing down ray clusters to cluster.apply() --- src/codeflare_sdk/ray/cluster/cluster.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/codeflare_sdk/ray/cluster/cluster.py b/src/codeflare_sdk/ray/cluster/cluster.py index b42efd87e..d03d78d8f 100644 --- a/src/codeflare_sdk/ray/cluster/cluster.py +++ b/src/codeflare_sdk/ray/cluster/cluster.py @@ -219,7 +219,7 @@ def apply(self, force=False): force_conflicts=force, ) print( - f"AppWrapper: '{name}' configuration has successfully been applied" + f"AppWrapper: '{name}' configuration has successfully been applied. For optimal resource management, you should delete this Ray Cluster when no longer in use." ) else: api_version = "ray.io/v1" @@ -227,7 +227,9 @@ def apply(self, force=False): self._component_resources_apply( namespace=namespace, api_instance=api_instance ) - print(f"Ray Cluster: '{name}' has successfully been applied") + print( + f"Ray Cluster: '{name}' has successfully been applied. For optimal resource management, you should delete this Ray Cluster when no longer in use." + ) except AttributeError as e: raise RuntimeError(f"Failed to initialize DynamicClient: {e}") except Exception as e: # pragma: no cover