Eirik A

Results 300 comments of Eirik A

Had a bit of a look at this in #65. I'm expecting we will want to generate something like: ```rust pub enum ServerPodSelector { MatchExpressions(Vec), MatchLabels(BTreeMap), } pub struct ServerPodSelectorMatchExpressions...

Ok, verified against kube-derive. The following example: ```rust use kube::{CustomResource, CustomResourceExt}; use schemars::JsonSchema; use std::collections::BTreeMap; use serde::{Deserialize, Serialize}; #[derive(CustomResource, Serialize, Deserialize, Debug, Clone, JsonSchema)] #[kube( group = "clux.dev", version =...

I think that's probably the current minimal if you want to use `k8s_openapi`'s [api feature](https://github.com/Arnavion/k8s-openapi/blob/4676156e69112713b8bf4b2e18b7e65a19e651a2/Cargo.toml#L52). The `#[kube]` instrumentation we provide in `kopium` is meant to simplify the use with [`kube::Api`](https://docs.rs/kube/latest/kube/struct.Api.html)...

I am curious about your `List` issue there as neither: - [k8s-openapi::List](https://docs.rs/k8s-openapi/0.15.0/k8s_openapi/struct.List.html) - [kube::core::object::ObjectList](https://docs.rs/kube/latest/kube/core/object/struct.ObjectList.html) has kind/version on the list wrapper struct, there's only `TypeMeta` on the elements inside `.items`. historically...

Thanks for the report. I am not sure what's best way forward right now. Your particular case does actually align with #31 and we can probably deal with it however...

hm, yeah agreed, that is probably a better default.

I had forgotten about this bug. I don't remember what we did in the last year and a half, but this no longer fails as can be seen in the...

@w3irdrobot i think we should be generating maps of string -> raw json in this case (since it's within an additionalProperties) what do you think of #244?

We actually won't need to `impl Default for SomeEnum` for enums because of the [rust 1.62 feature](https://blog.rust-lang.org/2022/06/30/Rust-1.62.0.html#default-enum-variants) allowing us to do: ```rust #[derive(Default)] enum Maybe { #[default] Nothing, Something(T), }...

Well, the test actually works even without repr (surprisingly), but turns out the enum isn't actually used in the schema because kopium just infers `i64` 🙃 ```rust #[derive(Serialize, Deserialize, Clone,...