On 25.08.2025 10:28, Wolfgang Bumiller wrote:
On Fri, Aug 22, 2025 at 11:00:38AM +0200, Gabriel Goller wrote:
+                let config = FabricConfig::parse_section_config(&raw_config)?;
+
+                let mut stats: Vec<NeighborStatus> = Vec::new();
+
+                for (nodeid, node) in config.values().flat_map(|entry| {
+                    entry
+                        .nodes()
+                        .map(|(id, node)| (id.to_string(), node.clone()))

^ ...
I'm sensing pattern here.

Would it make sense to add a `FabricConfig::all_nodes(&self) -> impl 
Iterator<...>` ?

Definitely makes sense, added it.

Also fixed all the other issues, thanks for the review!


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

Reply via email to