JSON patches not applying with patches, only patchesJson6902

[stuartpb]

See this commit in my system's repository. If I run kustomize build system/local before applying this change, all of the JSON patches are applied; if I switch the patchesJson6902: line to patches:, then several of the patches are not applied. Here's the diff between the two outputs:

25346c25346,25347
<     configInline: {}
---
>     configInline:
>       ingress-status-address: 192.168.42.80
25359a25361
>         st8s.testtrack4.com/zone: "403"
25367c25369,25370
<       service: {}
---
>       service:
>         loadBalancerIP: 192.168.42.80
25433a25437,25447
>     configInline: |
>       address-pools:
>       - name: household-static
>         protocol: layer2
>         addresses:
>         - 192.168.42.0 - 192.168.42.250
>         auto-assign: false
>       - name: household-dynamic
>         protocol: layer2
>         addresses:
>         - 192.168.32.0 - 192.168.32.250
25542a25557,25565
>         storageSpec:
>           volumeClaimTemplate:
>             spec:
>               accessModes:
>               - ReadWriteOnce
>               resources:
>                 requests:
>                   storage: 50Gi
>               storageClassName: throwaway
25670c25693,25694
<   labels: {}
---
>   labels:
>     st8s.testtrack4.com/zone: "403"
25676c25700,25701
<   labels: {}
---
>   labels:
>     st8s.testtrack4.com/zone: "403"
25682c25707,25708
<   labels: {}
---
>   labels:
>     st8s.testtrack4.com/zone: "403"
25688c25714,25716
<   labels: {}
---
>   labels:
>     st8s.testtrack4.com/storage-group: storberry
>     st8s.testtrack4.com/zone: "403"
25694c25722,25724
<   labels: {}
---
>   labels:
>     st8s.testtrack4.com/storage-group: storberry
>     st8s.testtrack4.com/zone: "403"
25700c25730,25733
<   labels: {}
---
>   labels:
>     st8s.testtrack4.com/hats: argon1
>     st8s.testtrack4.com/storage-group: storberry
>     st8s.testtrack4.com/zone: "403"

Kustomize version {Version:3.8.3 GitCommit:$Format:%H$ BuildDate:2020-09-18T18:54:15Z GoOs:linux GoArch:amd64}

This is on Arch Linux.

[Shell32-Natsu]

/assign

[Shell32-Natsu]

You have already had patches in the kustomization.yaml https://github.com/stuartpb/stubernetes/blob/da13816d6099750b7d56e11614495f70f82b1d57/system/local/kustomization.yaml#L95

Remove it will solve the issue.

[Shell32-Natsu]

/close

[k8s-ci-robot]

@Shell32-Natsu: Closing this issue.

In response to this:

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[stuartpb]

That's weird - shouldn't a YAML file with duplicated keys be rejected by kustomize?

[Shell32-Natsu]

This is the issue in go-yaml v2