use the closest region to the user, if it's not set

superfly · Aug 28, 2024 · 60b788a · 60b788a
1 parent 18a2a08
commit 60b788a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/internal/build/imgsrc/depot.go b/internal/build/imgsrc/depot.go
@@ -181,6 +181,13 @@ func initBuilder(ctx context.Context, buildState *build, appName string, streams
 		region = regionEnv
 	}
 
+	if region == "" {
+		closestRegion, err := apiClient.GetNearestRegion(ctx)
+		if err == nil {
+			region = closestRegion.Code
+		}
+	}
+
 	if region != "" {
 		region = "fly-" + region
 	}

diff --git a/internal/build/imgsrc/resolver.go b/internal/build/imgsrc/resolver.go
@@ -242,7 +242,7 @@ func (r *Resolver) BuildImage(ctx context.Context, streams *iostreams.IOStreams,
 		return nil, fmt.Errorf("invalid depot-scope value. must be 'org' or 'app'")
 	}
 
-	builderSize := flag.GetString(ctx, "depot-builder-size")
+	builderSize := flag.GetInt(ctx, "depot-builder-size")
 	region := flag.GetString(ctx, "depot-builder-region")
 	update := flag.GetBool(ctx, "depot-builder-update")