fix(sagemaker): add delay from start to connect and get spaces without interrupting execution (#8346)

## Problem
- User experiences pop up error when connecting with a stopped space.
- Attempt to connect without waiting for space to transition to running
state.
- Listing needs to check all clusters without stopping execution flow on
initial discovered error

## Solution
- Added delay for space to fully transition state before attempting to
connect
- Change throwing error to logging warnings

---

- Treat all work as PUBLIC. Private `feature/x` branches will not be
squash-merged at release time.
- Your code changes must meet the guidelines in
[CONTRIBUTING.md](https://github.com/aws/aws-toolkit-vscode/blob/master/CONTRIBUTING.md#guidelines).
- License: I confirm that my contribution is made under the terms of the
Apache 2.0 license.

---------

Co-authored-by: Laxman Reddy <[email protected]>

Author: aws-ajangg

packages/core/src/awsService/sagemaker/hyperpodCommands.ts

@@ -16,9 +16,30 @@ const localize = nls.loadMessageBundle()
 
 export async function openHyperPodRemoteConnection(node: SagemakerDevSpaceNode): Promise<void> {
     await startHyperpodSpaceCommand(node)
+    await waitForDevSpaceRunning(node)
     await connectToHyperPodDevSpace(node)
 }
 
+async function waitForDevSpaceRunning(node: SagemakerDevSpaceNode): Promise<void> {
+    const kubectlClient = node.getParent().getKubectlClient(node.hpCluster.clusterName)
+    if (!kubectlClient) {
+        getLogger().error(`No kubectlClient available for cluster: ${node.hpCluster.clusterName}`)
+        return
+    }
+    const timeout = 5 * 60 * 1000 // 5 minutes
+    const startTime = Date.now()
+
+    while (Date.now() - startTime < timeout) {
+        const status = await kubectlClient.getHyperpodSpaceStatus(node.devSpace)
+        if (status === 'Running') {
+            return
+        }
+        await new Promise((resolve) => setTimeout(resolve, 5000))
+    }
+
+    throw new Error('Timeout waiting for dev space to reach Running status')
+}
+
 export async function connectToHyperPodDevSpace(node: SagemakerDevSpaceNode): Promise<void> {
     const logger = getLogger()
 

packages/core/src/shared/clients/kubectlClient.ts

@@ -73,13 +73,13 @@ export class KubectlClient {
                 void vscode.window.showErrorMessage(
                     `You do not have permission to view ${eksCluster.name} or its spaces. Please contact your administrator.`
                 )
-                throw new Error(
-                    `Error: User has insufficient permissions to view EKS cluster (${eksCluster.name}) or its spaces.`
+                getLogger().warn(
+                    `[Warning]: User has insufficient permissions to view EKS cluster (${eksCluster.name}) or its spaces.`
                 )
             }
 
-            getLogger().error(
-                `Error: Unavailable spaces for EKS Cluster (${eksCluster.name}): ${error}\nStack trace: ${(error as Error).stack}`
+            getLogger().warn(
+                `[Warning]: Unavailable spaces for EKS Cluster (${eksCluster.name}): ${error}\nStack trace: ${(error as Error).stack}`
             )
         }
         return []