Improve error message clarity

maxwellgagnon · eywalker · commit e01b06974bb3 · 2025-10-26T17:26:03.000-07:00
diff --git a/internal/config/resources.go b/internal/config/resources.go
@@ -252,7 +252,7 @@ func (r *ResourceConfig) getCanonicalMemory() (int64, error) {
 // math.MaxInt64, it returns an overflow error.
 func bytesToMi(bytes float64) (int64, error) {
 	if bytes < 0 || math.IsNaN(bytes) || math.IsInf(bytes, 0) {
-		return 0, fmt.Errorf("memory must be >= 0")
+		return 0, fmt.Errorf("memory must be a valid, finite number >= 0.0 (not NaN or +/-Inf); got %g", bytes)
 	}
 	miFloat := bytes / (1024.0 * 1024.0)
 	if miFloat > float64(math.MaxInt64) {

Original file line number	Diff line number	Diff line change
`@@ -252,7 +252,7 @@ func (r *ResourceConfig) getCanonicalMemory() (int64, error) {`
`252`	`252`	`// math.MaxInt64, it returns an overflow error.`
`253`	`253`	`func bytesToMi(bytes float64) (int64, error) {`
`254`	`254`	`if bytes < 0 \|\| math.IsNaN(bytes) \|\| math.IsInf(bytes, 0) {`
`255`		`- return 0, fmt.Errorf("memory must be >= 0")`
	`255`	`+ return 0, fmt.Errorf("memory must be a valid, finite number >= 0.0 (not NaN or +/-Inf); got %g", bytes)`
`256`	`256`	`}`
`257`	`257`	`miFloat := bytes / (1024.0 * 1024.0)`
`258`	`258`	`if miFloat > float64(math.MaxInt64) {`