premkumar bhaskal

hmm understood, so you want to hide the complexity of overlay from end user, but still allow them to pass in the extra configurations. I think we can do that...

So i was checking this issue and this won't be fixed even if we use yaml v3 in ytt. There are open issues for exact problem in yaml project here...

@Xander-Polishchuk We will discuss this issue in our next community meeting to get input from experts. - I believe dropping yamlv3 for something else is definitely not an option right...

@oxfn the issue is with the underlying go/yaml library https://github.com/go-yaml/yaml/issues/435 , when a hex string is bigger than max integer representation in golang (for the int type), the parseInt fails...

@Xander-Polishchuk so we had discussed this issue in one of the community meetings and yes, we were considering if moving to https://github.com/kubernetes-sigs/yaml makes more sense now. Though this particular issue...

> go: carvel.dev/ytt/pkg/orderedmap tested by carvel.dev/ytt/pkg/orderedmap.test imports github.com/vmware-tanzu/carvel-ytt/pkg/orderedmap: no required module provides package github.com/vmware-tanzu/carvel-ytt/pkg/orderedmap; to add it: go get github.com/vmware-tanzu/carvel-ytt/pkg/orderedmap Error: Process completed with exit code 1. @notoriaga I think...

LGTM @100mik @cppforlife Please take a look.

will check this issue this week.

https://carvel.dev/ytt/#gist:https://gist.github.com/prembhaskal/626f1264d3ab6783d7c42ccf92eaeafd added a gist for simpler reproducing of issue.

few more illustrating similar issues - https://carvel.dev/ytt/#gist:https://gist.github.com/prembhaskal/e9fc5954c90eecd980c99533e3770070 - duplicated for all key not just matched - https://carvel.dev/ytt/#gist:https://gist.github.com/prembhaskal/cffd92d9678d4109d13d9f27b15e4e2a - duplicated for only cases where array was already empty.