Results 217 comments of Arnav Singh
trafficstars

I'll tweak the error to mention the possibility of multiple versions, and to suggest the `cargo tree` invocation.

Does the server allow creating a Secret with a `null` value? If yes, then rather than changing `ByteString::deserialize` to tolerate `null` the fix would be to change `Secret::data`'s type to...

The schema is https://github.com/Arnavion/k8s-openapi/blob/3024c3f25bc79825effab9e01c4a740793315928/k8s-openapi-codegen/src/supported_version.rs#L53-L60 , but my point about testing was not to look at the schema (which I already did, hence my note about `ConfigMap::binaryData` being the same) but...

Thanks. As one last test, can you check if `ConfigMap::data` has the same behavior? ie does it need to be fixed only for `{ "additionalProperties": { "type": "string", "format": "byte"...

It'll need to be treated like the regular `APIGroup` - https://github.com/Arnavion/k8s-openapi/blob/445e89ec444ebb1c68e61361e64eec4c4a3f4785/k8s-openapi-codegen-common/src/lib.rs#L452-L456

>Stuff for it is under [KEP 2155](https://github.com/kubernetes/enhancements/tree/master/keps/sig-api-machinery/2155-clientgo-apply) and some of my notes under [kube-rs/kube-rs#649](https://github.com/kube-rs/kube-rs/issues/649) Thanks for those links. For my reference: - https://github.com/kubernetes/kubernetes/blob/b8af116327cd5d8e5411cbac04e7d4d11d22485d/pkg/apis/apps/types.go#L135 vs https://github.com/kubernetes/kubernetes/blob/b8af116327cd5d8e5411cbac04e7d4d11d22485d/staging/src/k8s.io/client-go/applyconfigurations/apps/v1/statefulsetspec.go#L29 - https://github.com/kubernetes/kubernetes/commit/03d242665dbe39b3ceeb8562850602480e5788b8 + https://github.com/kubernetes/kubernetes/commit/e688f22da07465a88728210412470e14330f1563

Depends. Share your `AcmeCertManagerIo` CRD.

Also, if the only reason you're attempting this is to get CRUD API for your CRD, you may want to just use [k8s-openapi-derive](https://crates.io/crates/k8s-openapi-derive) instead. (Read the docs from [the source](https://github.com/Arnavion/k8s-openapi/blob/v0.6.0/k8s-openapi-derive/src/lib.rs#L35-L241)...

>I guess that `Nested anonymous types` codegen refers to is actually a listMeta with corresponding openapi schema: https://gist.github.com/ragne/ecebdbb09a29658c03624c5f03247683 I'm surprised that Kubernetes is inlining the defintion like that instead of...

>>I think one can try to construct the appropriate ListMeta struct dynamically or find already defined `ListMeta` or replace the literal definition with `$ref`. > >Replacing it with a `$ref`...