chore: Remove debug logging from stop/start operations

Removed temporary debug logging added during troubleshooting:
- Removed DEBUG print statements from StopContainerGroup
- Removed DEBUG/ERROR logging from stopWithACI
- Removed DEBUG/ERROR logging from stopWithACA
- Updated stopWithACA comment to reflect native Stop API usage

The stop/start functionality is now working correctly with:
- ACI: Using native client.Stop() API
- ACA: Using native client.BeginStop() API

Code is cleaner and production-ready without verbose debug output.

Author: VAIBHAVSING

apps/agent/internal/azure/client.go

@@ -330,14 +330,11 @@ func (c *Client) StopContainerGroup(ctx context.Context, region, resourceGroup,
 		return fmt.Errorf("failed to get ACI client for region %s: %w", region, err)
 	}
 
-	fmt.Printf("DEBUG: Attempting to stop container group: %s in resource group: %s, region: %s\n", name, resourceGroup, region)
-
 	_, err = client.Stop(ctx, resourceGroup, name, nil)
 	if err != nil {
 		return fmt.Errorf("failed to stop container group %s in resource group %s: %w", name, resourceGroup, err)
 	}
 
-	fmt.Printf("DEBUG: Successfully called Stop API for container group: %s\n", name)
 	return nil
 }
 

apps/agent/internal/services/deployment_strategy.go

@@ -296,27 +296,13 @@ func (d *DeploymentStrategy) deleteWithACA(ctx context.Context, workspaceID, res
 // stopWithACI stops a container using ACI (keeps it in stopped state)
 func (d *DeploymentStrategy) stopWithACI(ctx context.Context, workspaceID, region, resourceGroup string) error {
 	containerGroupName := fmt.Sprintf("aci-%s", workspaceID)
-	log.Printf("DEBUG: stopWithACI called for workspace %s, container group: %s", workspaceID, containerGroupName)
-	err := d.azureClient.StopContainerGroup(ctx, region, resourceGroup, containerGroupName)
-	if err != nil {
-		log.Printf("ERROR: stopWithACI failed for %s: %v", containerGroupName, err)
-	} else {
-		log.Printf("DEBUG: stopWithACI succeeded for %s", containerGroupName)
-	}
-	return err
+	return d.azureClient.StopContainerGroup(ctx, region, resourceGroup, containerGroupName)
 }
 
-// stopWithACA stops a container using ACA (scales to zero)
+// stopWithACA stops a container using ACA (uses native Stop API)
 func (d *DeploymentStrategy) stopWithACA(ctx context.Context, workspaceID, resourceGroup string) error {
 	containerAppName := fmt.Sprintf("aca-%s", workspaceID)
-	log.Printf("DEBUG: stopWithACA called for workspace %s, container app: %s", workspaceID, containerAppName)
-	err := d.azureClient.StopContainerApp(ctx, resourceGroup, containerAppName)
-	if err != nil {
-		log.Printf("ERROR: stopWithACA failed for %s: %v", containerAppName, err)
-	} else {
-		log.Printf("DEBUG: stopWithACA succeeded for %s (scaled to minReplicas=0, will stop when no traffic)", containerAppName)
-	}
-	return err
+	return d.azureClient.StopContainerApp(ctx, resourceGroup, containerAppName)
 }
 
 // startWithACI starts a container using ACI (starts stopped container or creates new one)