github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-25 12:43:23 +00:00
Code Issues Projects Releases Wiki Activity

Make NestedNumberAsFloat64 accuracy test architecture-neutral.

Browse Source 
NestedNumberAsFloat64 will convert int64s to float64 only if the int64 value can be represented
exactly by a float64. The original test for this property used a roundtrip conversion from int64 to
float64 and back, and the behavior of these conversions is inconsistent across architectures.
This commit is contained in:
Ben Luddy 2024-10-15 13:10:19 -04:00
parent 99cc395e2b
commit e413e026a3
No known key found for this signature in database
GPG Key ID: A6551E73A5974C30
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
staging/src/k8s.io/apimachinery/pkg/apis/meta/v1/unstructured/helpers.go
View File

@ -20,6 +20,7 @@ import (
gojson "encoding/json"
"fmt"
"io"
"math/big"
"strings"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
@ -136,10 +137,11 @@ func NestedNumberAsFloat64(obj map[string]interface{}, fields ...string) (float6
}
switch x := val.(type) {
case int64:
if x != int64(float64(x)) {
f, accuracy := big.NewInt(x).Float64()
if accuracy != big.Exact {
return 0, false, fmt.Errorf("%v accessor error: int64 value %v cannot be losslessly converted to float64", jsonPath(fields), x)
}
return float64(x), true, nil
return f, true, nil
case float64:
return x, true, nil
default: