applied this one early, thanks
On Wed, Jul 02, 2025 at 04:49:50PM +0200, Gabriel Goller wrote:
> From: Stefan Hanreich <s.hanre...@proxmox.com>
>
> The API macro required the enum variants to either have a oneOf or
> ObjectSchema, but did not allow allOf schemas. There's not really a
> reason to not allow allOf as well, since they implement
> ObjectSchemaType as well and represent an ObjectSchema, just like
> oneOf and ObjectSchema do.
>
> This is in preparation for the SDN fabrics, where sections use the
> allOf schema to merge general properties with protocol-specific
> properties.
>
> Signed-off-by: Stefan Hanreich <s.hanre...@proxmox.com>
> ---
> proxmox-api-macro/src/api/enums.rs | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/proxmox-api-macro/src/api/enums.rs
> b/proxmox-api-macro/src/api/enums.rs
> index 9b122f9c98ff..31a715dbf511 100644
> --- a/proxmox-api-macro/src/api/enums.rs
> +++ b/proxmox-api-macro/src/api/enums.rs
> @@ -253,6 +253,7 @@ fn handle_section_config_enum(
> match &<#ty as
> ::proxmox_schema::ApiType>::API_SCHEMA {
> ::proxmox_schema::Schema::Object(schema) =>
> schema,
> ::proxmox_schema::Schema::OneOf(schema) =>
> schema,
> + ::proxmox_schema::Schema::AllOf(schema) =>
> schema,
> _ => panic!("enum requires an object schema"),
> }
> }
> --
> 2.39.5
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
