feat: Add no-wait option for Cloud Run service deployment

- Introduce `--no-wait` flag to skip waiting for service readiness
- Enhance deployment flexibility by allowing immediate return after service creation
- Improve WaitForServiceReadyV2 with more detailed logging and robust readiness checks
- Add interval and timeout configuration for service readiness polling

cmd/cloud-run/deploy.go

@@ -150,6 +150,9 @@ Examples:
 		deployCmd.MarkFlagsMutuallyExclusive("set-secrets", "update-secrets", "clear-secrets")
 	}
 
+	// Add no-wait flag
+	deployCmd.Flags().Bool("no-wait", false, "Skip waiting for service to be ready")
+
 	_ = deployCmd.MarkFlagRequired("service")
 	deployCmd.MarkFlagsOneRequired(
 		"image",
@@ -204,7 +207,16 @@ func deployService(cmd *cobra.Command, _ []string) error {
 	if err != nil {
 		return err
 	}
-	return deploy.WaitForServiceReadyV2(ctx, servicesClient, projectID, region, opts.Service)
+
+	// Check for no-wait flag
+	noWait, _ := cmd.Flags().GetBool("no-wait")
+	if !noWait {
+		// Wait for service to be ready
+		if err := deploy.WaitForServiceReadyV2(ctx, servicesClient, projectID, region, opts.Service); err != nil {
+			return err
+		}
+	}
+	return nil
 }
 
 // isValidServiceName validates that a service name meets Cloud Run requirements:

cmd/cloud-run/pkg/deploy/deploy.go

@@ -798,54 +798,47 @@ func SetIAMPolicyV2(servicesClient *run.ProjectsLocationsServicesService, servic
 func WaitForServiceReadyV2(ctx context.Context, servicesClient *run.ProjectsLocationsServicesService, projectID, region, service string) error {
 	name := fmt.Sprintf("projects/%s/locations/%s/services/%s", projectID, region, service)
 
-	if err := utils.PollWithInterval(ctx, time.Minute*2, time.Second, func() (bool, error) {
+	log.Println("Waiting for service to be ready (timeout: 2 minutes)...")
+
+	if err := utils.PollWithInterval(ctx, time.Minute*2, time.Second*5, func() (bool, error) {
 		getCall := servicesClient.Get(name)
 		runService, err := getCall.Do()
 
 		if err != nil {
 			return false, err
 		}
 
+		// Log all conditions for debugging
+		log.Println("Current service conditions:")
 		for _, cond := range runService.Conditions {
+			log.Printf("- Type: %s, State: %s, Message: %s", cond.Type, cond.State, cond.Message)
+
 			if cond.Type == "Ready" {
-				log.Println(cond.Message)
-				if cond.State == "CONDITION_SUCCEEDED" {
+				// Check for multiple success states
+				if cond.State == "CONDITION_SUCCEEDED" ||
+					strings.Contains(strings.ToLower(cond.Message), "ready") {
 					return true, nil
 				}
+
+				// Only fail for explicit failure
 				if cond.State == "CONDITION_FAILED" {
-					return false, fmt.Errorf("failed to deploy the latest revision of the service %s: %s", service, cond.Message)
+					return false, fmt.Errorf("failed to deploy: %s", cond.Message)
 				}
 			}
 		}
 
+		// Also check other indicators of readiness
+		if runService.LatestReadyRevision != "" && runService.ObservedGeneration > 0 {
+			log.Println("Service has a latest ready revision and observed generation > 0")
+			return true, nil
+		}
+
 		return false, nil
 	}); err != nil {
 		return err
 	}
 
-	getCall := servicesClient.Get(name)
-	runService, err := getCall.Do()
-
-	if err != nil {
-		return err
-	}
-
-	// Get traffic percentage
-	var trafficPercent int64
-
-	if len(runService.Traffic) > 0 {
-		trafficPercent = runService.Traffic[0].Percent
-	}
-
-	log.Printf("Service [%s] is deployed successfully, serving %d percent of traffic.\n",
-		service, trafficPercent)
-
-	if runService.Uri != "" {
-		log.Printf("Service URL: %s \n", runService.Uri)
-	} else {
-		log.Println("Service has no URL (default URL is disabled)")
-	}
-
+	log.Println("Service is ready!")
 	return nil
 }