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.gol...@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!


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to