[pve-devel] [PATCH proxmox-ve-rs 12/17] ve-config: add openfabric section-config

Gabriel Goller g.goller at proxmox.com
Mon Mar 31 17:04:59 CEST 2025


On 31.03.2025 15:48, Christoph Heiss wrote:
>On Fri Mar 28, 2025 at 6:13 PM CET, Gabriel Goller wrote:
>> This is the main openfabric configuration. It is used to parse from the
>> section-config file (`/etc/pve/sdn/fabrics/openfabric.cfg`) and is also
>> returned from the api.
>>
>> Signed-off-by: Gabriel Goller <g.goller at proxmox.com>
>> ---
>>  proxmox-ve-config/Cargo.toml                  |   9 +-
>>  .../src/sdn/fabric/openfabric/mod.rs          | 291 ++++++++++++++++++
>>  2 files changed, 297 insertions(+), 3 deletions(-)
>>  create mode 100644 proxmox-ve-config/src/sdn/fabric/openfabric/mod.rs
>>
>> diff --git a/proxmox-ve-config/Cargo.toml b/proxmox-ve-config/Cargo.toml
>> index 4906d77550f3..3f7639efa153 100644
>> --- a/proxmox-ve-config/Cargo.toml
>> +++ b/proxmox-ve-config/Cargo.toml
>> @@ -10,14 +10,17 @@ exclude.workspace = true
>>  log = "0.4"
>>  anyhow = "1"
>>  nix = "0.26"
>> -thiserror = "1.0.59"
>> +thiserror = { workspace = true }
>>
>> -serde = { version = "1", features = [ "derive" ] }
>> +serde = { workspace = true, features = [ "derive" ] }
>> +serde_with = { workspace = true }
>>  serde_json = "1"
>>  serde_plain = "1"
>> -serde_with = "3"
>> +tracing = "0.1"
>>
>>  proxmox-schema = "4"
>> +proxmox-section-config = { workspace = true }
>> +proxmox-serde = { version = "0.1.2" }
>>  proxmox-sys = "0.6.4"
>>  proxmox-sortable-macro = "0.1.3"
>>  proxmox-network-types = { version = "0.1", path = "../proxmox-network-types/" }
>
>nit: These changes should be reflected in debian/control too.
>
>Its sync'd up later in patch #16 ("ve-config: add section-config to frr
>types conversion") as part of another change in Cargo.toml, but they
>should always be grouped together, IMO.

Oops, yeah, will fix this.
Also messed up the similar-asserts import and debian/control entry as in
your other reply.

Also changed from similar-asserts to insta, so this shouldn't be an
issue in the next version!

Thanks for reviewing!




More information about the pve-devel mailing list