fix: use k8syml.YAMLToJSON when marshalling resources

Properly handles cases where the YAML is not valid JSON, such as
maps with integer keys (#3446).

api/krusty/configmaps_test.go

@@ -312,6 +312,31 @@ metadata:
 `)
 }
 
+func TestIssue3446(t *testing.T) {
+ th := kusttest_test.MakeHarness(t)
+ th.WriteK(".", `
+resources:
+- cm.yaml
+`)
+ th.WriteF("cm.yaml", `
+apiVersion: v1
+kind: ConfigMap
+metadata:
+ name: cm
+data:
+ 123: abc
+`)
+ m := th.Run(".", th.MakeDefaultOptions())
+ th.AssertActualEqualsExpected(m, `
+apiVersion: v1
+data:
+ "123": abc
+kind: ConfigMap
+metadata:
+ name: cm
+`)
+}
+
 func TestGeneratorSimpleOverlay(t *testing.T) {
  th := kusttest_test.MakeHarness(t)
  th.WriteK("base", `

kyaml/yaml/rnode.go

@@ -12,6 +12,8 @@ import (
  "strconv"
  "strings"
 
+ k8syaml "sigs.k8s.io/yaml"
+
  "sigs.k8s.io/kustomize/kyaml/errors"
  "sigs.k8s.io/kustomize/kyaml/internal/forked/github.com/go-yaml/yaml"
  "sigs.k8s.io/kustomize/kyaml/sliceutil"
@@ -887,11 +889,11 @@ func (rn *RNode) MarshalJSON() ([]byte, error) {
  return json.Marshal(a)
  }
 
- m := map[string]interface{}{}
- if err := Unmarshal([]byte(s), &m); err != nil {
- return nil, err
+ b, err := k8syaml.YAMLToJSON([]byte(s))
+ if err != nil {
+ return nil, errors.Wrap(err)
  }
- return json.Marshal(m)
+ return b, nil
 }
 
 // UnmarshalJSON overwrites this RNode with data from []byte.