Consider switching from PyYAML to ruamel.yaml?

[XeCycle]

PyYAML implements YAML 1.1, and ruamel.yaml implements 1.2. It seems to me that kubectl accepts 1.2. For example:

dump_all_yaml([ConfigMap(data={"hello": "012345678"})], sys.stdout)

produces

data:
  hello: 012345678

and that is parsed as number, and will be rejected by the server because of type mismatch.

[gtsystem]

According to the documentation, "..Kubernetes uses YAML parsers that are mostly compatible with YAML 1.1..". So it's some sort of hibrid between 1.1 and 1.2. See also kubernetes/kubernetes#34146.

However it would make sense to support 1.2, but we cannot really break existing users expecting that the parser uses 1.1.
So my suggestion would be to add a parameter to codecs.load_all_yaml and codecs.dump_all_yaml with default "1.1" and optional value "1.2". We can switch to ruamel.yaml that support both versions.

Would it work for you? Do you want to create a PR?

[XeCycle]

Oh so k8s parsers are neither 1.1 nor 1.2 😂 didn't know that. In my case I actually stopped using yaml from lightkube so I won't be affected anyway, but this situation looks nasty I can't come up with an easy solution, other than using directly the go libraries. Maybe it's better to leave it unchanged, or add some documentation on this problem.