[pve-devel] [PATCH proxmox-ve-rs 03/11] add intermediate fabric representation
Gabriel Goller
g.goller at proxmox.com
Fri Feb 28 17:19:20 CET 2025
On 28.02.2025 14:57, Thomas Lamprecht wrote:
>Am 14.02.25 um 14:39 schrieb Gabriel Goller:
>> This adds the intermediate, type-checked fabrics config. This one is
>> parsed from the SectionConfig and can be converted into the
>> Frr-Representation.
>
>The short description of the patch is good, but I would like to see more
>rationale here about choosing this way, like benefits and trade-offs to other
>options that got evaluated, if this can/will be generic for all fabrics planned,
>..., and definitively some more rust-documentation for public types and modules.
Yep, wrote together a small reasoning and will write some more
documentation for public types.
>One thing I noticed below, I did not managed to do a thorough review besides
>of that yet though.
>> [snip]
>> +impl Hostname {
>> + pub fn new(name: impl Into<String>) -> Hostname {
>> + Self(name.into())
>> + }
>> +}
>> +
>> +// parses a bool from a string OR bool
>> +pub mod serde_option_bool {
>
>might be maybe something to put in proxmox-serde?
Yep, I agree.
Thanks for the review!
More information about the pve-devel
mailing list