Re: [PATCH pve-network 06/16] evpn controller: allow multiple evpn controllers in a cluster

Stefan Hanreich
2026-04-16T15:38:55Z

On 4/16/26 5:32 PM, Gabriel Goller wrote:
> On 16.04.2026 17:30, Hannes Laimer wrote:
>> On 2026-04-14 18:33, Stefan Hanreich wrote:
>> if one has none, so the default('VTEP'), and the other sets 'VTEP', we
>> miss that case here I think. something like
>> ```
>> my $pg_self = $controller->{'peer-group-name'} // 'VTEP';
>> my $pg_other = $other_controller->{'peer-group-name'} // 'VTEP';
>> die "cannot have two controllers with ..." if $pg_self eq $pg_other;
>> ```
>> might be better
>
> Maybe tangentially related: we should probably disallow VTEP as a bgp fabric
> name.

good catch! i'll fix it in the next version!