Update internal/clouds/general/location.go

aronchick · coderabbitai[bot] · web-flow · commit b37ef741e51f · 2024-12-14T10:16:02.000-08:00
Co-authored-by: coderabbitai[bot] &lt;136622811+coderabbitai[bot]@users.noreply.github.com&gt;
diff --git a/internal/clouds/general/location.go b/internal/clouds/general/location.go
@@ -57,7 +57,7 @@ func NormalizeLocation(
 		return region, zone, nil
 	}
 	l.Warnf("Unknown cloud provider: %s", cloudProvider)
-	return location, location, nil
+	return "", "", fmt.Errorf("unknown cloud provider: %s", cloudProvider)
 }
 
 // ParseRegionZone takes either a region (e.g., us-east-1) or zone (e.g., us-east-1a)

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ func NormalizeLocation(`
`57`	`57`	`return region, zone, nil`
`58`	`58`	`}`
`59`	`59`	`l.Warnf("Unknown cloud provider: %s", cloudProvider)`
`60`		`- return location, location, nil`
	`60`	`+ return "", "", fmt.Errorf("unknown cloud provider: %s", cloudProvider)`
`61`	`61`	`}`
`62`	`62`
`63`	`63`	`// ParseRegionZone takes either a region (e.g., us-east-1) or zone (e.g., us-east-1a)`