public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Gabriel Goller <g.goller@proxmox.com>
To: Wolfgang Bumiller <w.bumiller@proxmox.com>
Cc: pve-devel@lists.proxmox.com
Subject: Re: [pve-devel] [PATCH proxmox v4 1/5] network-types: initial commit
Date: Fri, 4 Jul 2025 14:19:45 +0200	[thread overview]
Message-ID: <mjlcrzzoiqwrpxuwkyb2br3mlvihedyfw6bghed5eonaovt7ng@6hkt6muloyy3> (raw)
In-Reply-To: <rymlert6hm4xcwhydgp2ur25aehznutdicvkkbc2l6t6bndcm6@d4plmziszjif>

>> [snip]
>>  # workspace dependencies
>> -proxmox-access-control = { version = "0.2.5", path = "proxmox-access-control" }
>> -proxmox-acme = {  version = "1.0.0", path = "proxmox-acme", default-features = false }
>> -proxmox-api-macro = { version = "1.4.0", path = "proxmox-api-macro" }
>> -proxmox-apt-api-types = { version = "2.0.0", path = "proxmox-apt-api-types" }
>> -proxmox-auth-api = { version = "1.0.0", path = "proxmox-auth-api" }
>> -proxmox-async = { version = "0.5.0", path = "proxmox-async" }
>> -proxmox-base64 = {  version = "1.0.0", path = "proxmox-base64" }
>> -proxmox-compression = { version = "1.0.0", path = "proxmox-compression" }
>> -proxmox-daemon = { version = "1.0.0", path = "proxmox-daemon" }
>> -proxmox-http = { version = "1.0.0", path = "proxmox-http" }
>> -proxmox-http-error = { version = "1.0.0", path = "proxmox-http-error" }
>> -proxmox-human-byte = { version = "1.0.0", path = "proxmox-human-byte" }
>> -proxmox-io = { version = "1.2.0", path = "proxmox-io" }
>> -proxmox-lang = { version = "1.5", path = "proxmox-lang" }
>> -proxmox-log = { version = "1.0.0", path = "proxmox-log" }
>> -proxmox-login = { version = "1.0.0", path = "proxmox-login" }
>> -proxmox-product-config = { version = "1.0.0", path = "proxmox-product-config" }
>> -proxmox-config-digest = { version = "1.0.0", path = "proxmox-config-digest" }
>> -proxmox-rest-server = { version = "1.0.0", path = "proxmox-rest-server" }
>> -proxmox-router = { version = "3.2.2", path = "proxmox-router" }
>> -proxmox-schema = { version = "4.1.0", path = "proxmox-schema" }
>> -proxmox-section-config = { version = "3.1.0", path = "proxmox-section-config" }
>> -proxmox-sendmail = { version = "1.0.0", path = "proxmox-sendmail" }
>> -proxmox-serde = { version = "1.0.0", path = "proxmox-serde", features = [ "serde_json" ] }
>> -proxmox-shared-memory = { version = "1.0.0", path = "proxmox-shared-memory" }
>> +proxmox-acme = {  version = "0.5.3", path = "proxmox-acme", default-features = false }
>> +proxmox-api-macro = { version = "1.3.2", path = "proxmox-api-macro" }
>> +proxmox-apt-api-types = { version = "1.0.2", path = "proxmox-apt-api-types" }
>> +proxmox-auth-api = { version = "0.4.0", path = "proxmox-auth-api" }
>> +proxmox-async = { version = "0.4.1", path = "proxmox-async" }
>> +proxmox-compression = { version = "0.2.4", path = "proxmox-compression" }
>> +proxmox-daemon = { version = "0.1.0", path = "proxmox-daemon" }
>> +proxmox-http = { version = "0.9.5", path = "proxmox-http" }
>> +proxmox-http-error = { version = "0.1.0", path = "proxmox-http-error" }
>> +proxmox-human-byte = { version = "0.1.0", path = "proxmox-human-byte" }
>> +proxmox-io = { version = "1.1.0", path = "proxmox-io" }
>> +proxmox-lang = { version = "1.3", path = "proxmox-lang" }
>> +proxmox-log= { version = "0.2.9", path = "proxmox-log" }
>> +proxmox-login = { version = "0.2.0", path = "proxmox-login" }
>> +proxmox-network-types = { version = "0.1.0", path = "proxmox-network-types" }
>> +proxmox-product-config = { version = "0.2.0", path = "proxmox-product-config" }
>> +proxmox-config-digest = { version = "0.1.0", path = "proxmox-config-digest" }
>> +proxmox-rest-server = { version = "0.8.8", path = "proxmox-rest-server" }
>> +proxmox-router = { version = "3.1.1", path = "proxmox-router" }
>> +proxmox-schema = { version = "4.0.0", path = "proxmox-schema" }
>> +proxmox-section-config = { version = "3.0.0", path = "proxmox-section-config" }
>> +proxmox-sendmail = { version = "0.1.0", path = "proxmox-sendmail" }
>> +proxmox-serde = { version = "0.1.1", path = "proxmox-serde", features = [ "serde_json" ] }
>> +proxmox-shared-memory = { version = "0.3.0", path = "proxmox-shared-memory" }
>>  proxmox-sortable-macro = { version = "0.1.3", path = "proxmox-sortable-macro" }
>
>^ Reverts to bookworm deps - you sure the patches are otherwise based on
>trixie?

Missed this hunk, my bad. The rest should be rebased on trixie
correctly.

>> diff --git a/proxmox-network-types/src/ip_address.rs b/proxmox-network-types/src/ip_address.rs
>> new file mode 100644
>> index 000000000000..355547b17ae0
>> --- /dev/null
>> +++ b/proxmox-network-types/src/ip_address.rs
>> @@ -0,0 +1,1410 @@
>> +//! Provides helpers to deal with IP addresses / CIDRs
>> +
>> +use std::net::{AddrParseError, IpAddr, Ipv4Addr, Ipv6Addr};
>> +
>> +use serde_with::{DeserializeFromStr, SerializeDisplay};
>> +use thiserror::Error;
>> +
>> +/// The family (v4 or v6)  of an IP address or CIDR prefix
>> +#[derive(Clone, Copy, Debug, Eq, PartialEq)]
>> +pub enum Family {
>> +    V4,
>> +    V6,
>> +}
>> +
>> +impl Family {
>> +    pub fn is_ipv4(&self) -> bool {
>> +        *self == Self::V4
>> +    }
>
>nit: ↑↓ those could take self copied since it's Family is Copy.

Done.

>> +
>> +    pub fn is_ipv6(&self) -> bool {
>> +        *self == Self::V6
>> +    }
>> +}
>> +
>> +impl std::fmt::Display for Family {
>> +    fn fmt(&self, f: &mut std::fmt::Formatter) -> std::fmt::Result {
>> +        match self {
>> +            Family::V4 => f.write_str("Ipv4"),
>> +            Family::V6 => f.write_str("Ipv6"),
>
>For a Display implementation it IMO makes no sense to mix
>capital/lowercase in "IP". If we need to keep this for ser/de purposes,
>add a comment please. Otherwise change it to IPv4/IPv6.

Agree.

>> +        }
>> +    }
>> +}
>> +
>> +#[derive(Error, Debug)]
>> +pub enum CidrError {
>> +    #[error("invalid netmask")]
>> +    InvalidNetmask,
>> +    #[error("invalid IP address")]
>> +    InvalidAddress(#[from] AddrParseError),
>> +}
>> +
>> +/// Represents either an [`Ipv4Cidr`] or [`Ipv6Cidr`] CIDR prefix
>> +#[derive(
>> +    Clone, Copy, Debug, PartialOrd, Ord, PartialEq, Eq, Hash, SerializeDisplay, DeserializeFromStr,
>> +)]
>> +pub enum Cidr {
>> +    Ipv4(Ipv4Cidr),
>> +    Ipv6(Ipv6Cidr),
>> +}
>> +
>> +impl Cidr {
>> +    pub fn new_v4(addr: impl Into<Ipv4Addr>, mask: u8) -> Result<Self, CidrError> {
>> +        Ok(Cidr::Ipv4(Ipv4Cidr::new(addr, mask)?))
>> +    }
>> +
>> +    pub fn new_v6(addr: impl Into<Ipv6Addr>, mask: u8) -> Result<Self, CidrError> {
>> +        Ok(Cidr::Ipv6(Ipv6Cidr::new(addr, mask)?))
>> +    }
>> +
>> +    /// which [`Family`] this CIDR belongs to
>> +    pub const fn family(&self) -> Family {
>> +        match self {
>> +            Cidr::Ipv4(_) => Family::V4,
>> +            Cidr::Ipv6(_) => Family::V6,
>> +        }
>> +    }
>> +
>> +    pub fn is_ipv4(&self) -> bool {
>> +        matches!(self, Cidr::Ipv4(_))
>> +    }
>> +
>> +    pub fn is_ipv6(&self) -> bool {
>> +        matches!(self, Cidr::Ipv6(_))
>> +    }
>> +
>> +    /// Whether a given IP address is contained in this [`Cidr`]
>> +    ///
>> +    /// This only works if both [`IpAddr`] are in the same family, otherwise the function returns
>> +    /// false.
>> +    pub fn contains_address(&self, ip: &IpAddr) -> bool {
>> +        match (self, ip) {
>> +            (Cidr::Ipv4(cidr), IpAddr::V4(ip)) => cidr.contains_address(ip),
>> +            (Cidr::Ipv6(cidr), IpAddr::V6(ip)) => cidr.contains_address(ip),
>> +            _ => false,
>> +        }
>> +    }
>> +}
>> +
>> +impl std::fmt::Display for Cidr {
>> +    fn fmt(&self, f: &mut std::fmt::Formatter) -> std::fmt::Result {
>> +        match self {
>> +            Self::Ipv4(ip) => f.write_str(ip.to_string().as_str()),
>> +            Self::Ipv6(ip) => f.write_str(ip.to_string().as_str()),
>
>No need to allocate a new string if you just forward to the inner value,
>either use `write!(f, ip)` (more convenient), or explicitly forward via
>`Display::fmt(ip, f)`:
>
>            Self::Ipv4(ip) => fmt::Display::fmt(ip, f),
>            Self::Ipv6(ip) => fmt::Display::fmt(ip, f),
>

Would have been `write!(f, "{}", ip)`, so I went with Display::fmt :)

>> +        }
>> +    }
>> +}
>> +
>> +impl std::str::FromStr for Cidr {
>> +    type Err = CidrError;
>> +
>> +    fn from_str(s: &str) -> Result<Self, Self::Err> {
>> +        if let Ok(ip) = s.parse::<Ipv4Cidr>() {
>> +            return Ok(Cidr::Ipv4(ip));
>> +        }
>> +
>> +        Ok(Cidr::Ipv6(s.parse()?))
>> +    }
>> +}
>> +
>> +impl From<Ipv4Cidr> for Cidr {
>> +    fn from(cidr: Ipv4Cidr) -> Self {
>> +        Cidr::Ipv4(cidr)
>> +    }
>> +}
>> +
>> +impl From<Ipv6Cidr> for Cidr {
>> +    fn from(cidr: Ipv6Cidr) -> Self {
>> +        Cidr::Ipv6(cidr)
>> +    }
>> +}
>> +
>> +impl From<IpAddr> for Cidr {
>> +    fn from(value: IpAddr) -> Self {
>> +        match value {
>> +            IpAddr::V4(addr) => Ipv4Cidr::from(addr).into(),
>> +            IpAddr::V6(addr) => Ipv6Cidr::from(addr).into(),
>> +        }
>> +    }
>> +}
>> +
>> +const IPV4_LENGTH: u8 = 32;
>> +
>> +/// An IPv4 CIDR (e.g. 192.0.2.0/24)
>> +#[derive(
>> +    SerializeDisplay, DeserializeFromStr, Clone, Copy, Debug, PartialOrd, Ord, PartialEq, Eq, Hash,
>
>nit: would prefer the standard stuff first

Agree

>> [snip]
>> +impl std::str::FromStr for Ipv4Cidr {
>> +    type Err = CidrError;
>> +
>> +    fn from_str(s: &str) -> Result<Self, Self::Err> {
>> +        Ok(match s.find('/') {
>> +            None => Self {
>> +                addr: s.parse()?,
>> +                mask: 32,
>> +            },
>> +            Some(pos) => {
>> +                let mask: u8 = s[(pos + 1)..]
>> +                    .parse()
>> +                    .map_err(|_| CidrError::InvalidNetmask)?;
>> +
>> +                Self::new(s[..pos].parse::<Ipv4Addr>()?, mask)?
>> +            }
>> +        })
>
>^ since 1.52 this could be more readable with `.split_once('/')`, then
>we don't need to slice manually with `..pos`/`(pos + 1)..`.

Fixed this.

>> +    }
>> +}
>> +
>> +impl std::fmt::Display for Ipv4Cidr {
>> +    fn fmt(&self, f: &mut std::fmt::Formatter) -> std::fmt::Result {
>> +        write!(f, "{}/{}", &self.addr, self.mask)
>
>^ unnecessary &

Agree.

>> [snip]
>> +#[cfg(test)]
>> +mod tests {
>> +    use super::*;
>> +    use std::net::{Ipv4Addr, Ipv6Addr};
>
>^ std import should be grouped befure the `super::*` one.

cargo fmt puts super over std :(

Thanks for the review!


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

  parent reply	other threads:[~2025-07-04 12:19 UTC|newest]

Thread overview: 129+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-07-02 14:49 [pve-devel] [PATCH access-control/cluster/docs/gui-tests/manager/network/proxmox{, -firewall, -ve-rs, -perl-rs, -widget-toolkit} v4 00/76] Add SDN Fabrics Gabriel Goller
2025-07-02 14:49 ` [pve-devel] [PATCH proxmox v4 1/5] network-types: initial commit Gabriel Goller
2025-07-03 13:11   ` Wolfgang Bumiller
2025-07-03 13:46     ` Stefan Hanreich
2025-07-03 13:48       ` Stefan Hanreich
2025-07-04  7:55         ` Wolfgang Bumiller
2025-07-04 12:19     ` Gabriel Goller [this message]
2025-07-02 14:49 ` [pve-devel] [PATCH proxmox v4 2/5] network-types: make cidr and mac-address types usable by the api Gabriel Goller
2025-07-02 14:49 ` [pve-devel] [PATCH proxmox v4 3/5] network-types: add api types for ipv4/6 Gabriel Goller
2025-07-03 13:18   ` Wolfgang Bumiller
2025-07-02 14:49 ` [pve-devel] [PATCH proxmox v4 4/5] network-types: add CIDR overlap detection for IPv4 and IPv6 Gabriel Goller
2025-07-02 14:49 ` [pve-devel] [PATCH proxmox v4 5/5] api-macro: add allof schema to enum Gabriel Goller
2025-07-04 13:48   ` [pve-devel] applied: " Wolfgang Bumiller
2025-07-02 14:49 ` [pve-devel] [PATCH proxmox-firewall v4 1/1] firewall: nftables: migrate to proxmox-network-types Gabriel Goller
2025-07-02 14:49 ` [pve-devel] [PATCH proxmox-ve-rs v4 01/22] config: use proxmox_serde perl helpers Gabriel Goller
2025-07-04 14:09   ` [pve-devel] applied: " Wolfgang Bumiller
2025-07-02 14:49 ` [pve-devel] [PATCH proxmox-ve-rs v4 02/22] ve-config: move types to proxmox-network-types Gabriel Goller
2025-07-02 14:49 ` [pve-devel] [PATCH proxmox-ve-rs v4 03/22] sdn-types: initial commit Gabriel Goller
2025-07-04 14:09   ` Wolfgang Bumiller
2025-07-04 14:40     ` Gabriel Goller
2025-07-07  7:53       ` Wolfgang Bumiller
2025-07-02 14:49 ` [pve-devel] [PATCH proxmox-ve-rs v4 04/22] frr: create proxmox-frr crate Gabriel Goller
2025-07-07 11:12   ` Wolfgang Bumiller
2025-07-07 12:52     ` Gabriel Goller
2025-07-02 14:49 ` [pve-devel] [PATCH proxmox-ve-rs v4 05/22] frr: add common frr types Gabriel Goller
2025-07-07 11:18   ` Wolfgang Bumiller
2025-07-07 14:43     ` Gabriel Goller
2025-07-02 14:49 ` [pve-devel] [PATCH proxmox-ve-rs v4 06/22] frr: add openfabric types Gabriel Goller
2025-07-07 11:25   ` Wolfgang Bumiller
2025-07-07 15:19     ` Gabriel Goller
2025-07-02 14:49 ` [pve-devel] [PATCH proxmox-ve-rs v4 07/22] frr: add ospf types Gabriel Goller
2025-07-07 11:28   ` Wolfgang Bumiller
2025-07-07 17:07     ` Gabriel Goller
2025-07-02 14:49 ` [pve-devel] [PATCH proxmox-ve-rs v4 08/22] frr: add route-map types Gabriel Goller
2025-07-07 11:52   ` Wolfgang Bumiller
2025-07-07 17:13     ` Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-ve-rs v4 09/22] frr: add generic types over openfabric and ospf Gabriel Goller
2025-07-07 11:58   ` Wolfgang Bumiller
2025-07-07 17:20     ` Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-ve-rs v4 10/22] config: sdn: fabrics: add section types Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-ve-rs v4 11/22] config: sdn: fabrics: add node " Gabriel Goller
2025-07-07 12:19   ` Wolfgang Bumiller
2025-07-07 17:22     ` Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-ve-rs v4 12/22] config: sdn: fabrics: add interface name struct Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-ve-rs v4 13/22] config: sdn: fabrics: add openfabric properties Gabriel Goller
2025-07-08  8:09   ` Wolfgang Bumiller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-ve-rs v4 14/22] config: sdn: fabrics: add ospf properties Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-ve-rs v4 15/22] config: sdn: fabrics: add api types Gabriel Goller
2025-07-08  8:15   ` Wolfgang Bumiller
2025-07-08  9:50     ` Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-ve-rs v4 16/22] config: sdn: fabrics: add section config Gabriel Goller
2025-07-08  8:18   ` Wolfgang Bumiller
2025-07-11  8:33     ` Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-ve-rs v4 17/22] config: sdn: fabrics: add fabric config Gabriel Goller
2025-07-08  8:46   ` Wolfgang Bumiller
2025-07-11  9:04     ` Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-ve-rs v4 18/22] common: sdn: fabrics: implement validation Gabriel Goller
2025-07-08 11:01   ` Wolfgang Bumiller
2025-07-11  9:40     ` Gabriel Goller
2025-07-11 11:38       ` Wolfgang Bumiller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-ve-rs v4 19/22] sdn: fabrics: config: add conversion from / to section config Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-ve-rs v4 20/22] sdn: fabrics: implement FRR configuration generation Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-ve-rs v4 21/22] ve-config: add integrations tests Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-ve-rs v4 22/22] frr: add global ipv6 forwarding Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-perl-rs v4 1/5] pve-rs: Add PVE::RS::SDN::Fabrics module Gabriel Goller
2025-07-04 12:16   ` Wolfgang Bumiller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-perl-rs v4 2/5] pve-rs: sdn: fabrics: add api methods Gabriel Goller
2025-07-04 12:57   ` Wolfgang Bumiller
2025-07-04 15:56     ` Gabriel Goller
2025-07-07  7:48       ` Wolfgang Bumiller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-perl-rs v4 3/5] pve-rs: sdn: fabrics: add frr config generation Gabriel Goller
2025-07-04 13:14   ` Wolfgang Bumiller
2025-07-04 13:23     ` Stefan Hanreich
2025-07-04 13:31       ` Wolfgang Bumiller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-perl-rs v4 4/5] pve-rs: sdn: fabrics: add helper to generate ifupdown2 configuration Gabriel Goller
2025-07-04 13:29   ` Wolfgang Bumiller
2025-07-04 16:15     ` Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-perl-rs v4 5/5] pve-rs: sdn: fabrics: add helper for network API endpoint Gabriel Goller
2025-07-04 13:34   ` Wolfgang Bumiller
2025-07-04 16:31     ` Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-cluster v4 1/1] cfs: add fabrics.cfg to observed files Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-access-control v4 1/1] permissions: add ACL paths for SDN fabrics Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 01/21] sdn: fix value returned by pending_config Gabriel Goller
2025-07-04 14:29   ` Stefan Hanreich
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 02/21] debian: add dependency to proxmox-perl-rs Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 03/21] fabrics: add fabrics module Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 04/21] refactor: controller: move frr methods into helper Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 05/21] frr: add new helpers for reloading frr configuration Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 06/21] controllers: define new api for frr config generation Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 07/21] sdn: add frr config generation helpers Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 08/21] sdn: api: add check for rewriting frr configuration Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 09/21] test: isis: add test for standalone configuration Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 10/21] sdn: frr: add daemon status to frr helper Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 11/21] sdn: commit fabrics config to running configuration Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 12/21] fabrics: generate ifupdown configuration Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 13/21] fabrics: add jsonschema for fabrics and nodes Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 14/21] api: fabrics: add root-level module Gabriel Goller
2025-07-04 14:32   ` Stefan Hanreich
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 15/21] api: fabrics: add fabric submodule Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 16/21] api: fabrics: add node submodule Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 17/21] api: fabrics: add fabricnode submodule Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 18/21] controller: evpn: add fabrics integration Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 19/21] zone: vxlan: " Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 20/21] test: fabrics: add test cases for ospf and openfabric + evpn Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-network v4 21/21] frr: bump frr config version to 10.3.1 Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH proxmox-widget-toolkit v4 1/1] network selector: add type parameter Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 01/17] api: use new sdn config generation functions Gabriel Goller
2025-07-04 14:27   ` Stefan Hanreich
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 02/17] ui: fabrics: add model definitions for fabrics Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 03/17] fabric: add common interface panel Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 04/17] fabric: add OpenFabric interface properties Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 05/17] fabric: add OSPF " Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 06/17] fabric: add generic node edit panel Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 07/17] fabric: add OpenFabric node edit Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 08/17] fabric: add OSPF " Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 09/17] fabric: add generic fabric edit panel Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 10/17] fabric: add OpenFabric " Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 11/17] fabric: add OSPF " Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 12/17] fabrics: Add main FabricView Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 13/17] ui: permissions: add ACL path for fabrics Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 14/17] api: network: add include_sdn / fabric type Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 15/17] ui: add sdn networks to ceph / migration Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 16/17] ui: sdn: add evpn controller fabric integration Gabriel Goller
2025-07-02 14:50 ` [pve-devel] [PATCH pve-manager v4 17/17] ui: sdn: vxlan: add fabric property Gabriel Goller
2025-07-02 14:51 ` [pve-devel] [PATCH pve-gui-tests v4 1/1] pve: add sdn/fabrics screenshots Gabriel Goller
2025-07-02 14:51 ` [pve-devel] [PATCH pve-docs v4 1/1] fabrics: add initial documentation for sdn fabrics Gabriel Goller
2025-07-03  9:57 ` [pve-devel] [PATCH access-control/cluster/docs/gui-tests/manager/network/proxmox{, -firewall, -ve-rs, -perl-rs, -widget-toolkit} v4 00/76] Add SDN Fabrics Gabriel Goller
2025-07-09 11:27 ` Gabriel Goller
2025-07-16 13:09   ` Gabriel Goller

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=mjlcrzzoiqwrpxuwkyb2br3mlvihedyfw6bghed5eonaovt7ng@6hkt6muloyy3 \
    --to=g.goller@proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    --cc=w.bumiller@proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal