From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: <pve-devel-bounces@lists.proxmox.com> Received: from firstgate.proxmox.com (firstgate.proxmox.com [IPv6:2a01:7e0:0:424::9]) by lore.proxmox.com (Postfix) with ESMTPS id 33DE61FF189 for <inbox@lore.proxmox.com>; Fri, 4 Apr 2025 09:53:38 +0200 (CEST) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 7083818526; Fri, 4 Apr 2025 09:53:24 +0200 (CEST) Message-ID: <3a9e9a84-cbf3-4958-b5be-31bcf10d72df@proxmox.com> Date: Fri, 4 Apr 2025 09:53:21 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird To: Friedrich Weber <f.weber@proxmox.com>, Proxmox VE development discussion <pve-devel@lists.proxmox.com>, Gabriel Goller <g.goller@proxmox.com> References: <20250328171340.885413-1-g.goller@proxmox.com> <c3d5e091-7a92-4f2f-be6a-4753b5691492@proxmox.com> <dkweqizdmmrc3pf7n7z663eia7dweslqmd3tl6maqi3lg2jokx@sa4rn2slkqq6> <cf52115c-ee4c-4927-817f-9ba202214f3a@proxmox.com> <b093f56f-c445-45da-a11a-998030f3a972@proxmox.com> <6cd7cf05-443a-4fc5-afb3-4d2b5f7334bc@proxmox.com> Content-Language: en-US From: Stefan Hanreich <s.hanreich@proxmox.com> In-Reply-To: <6cd7cf05-443a-4fc5-afb3-4d2b5f7334bc@proxmox.com> X-SPAM-LEVEL: Spam detection results: 0 AWL 0.670 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Subject: Re: [pve-devel] [PATCH cluster/docs/manager/network/proxmox{, -ve-rs, -firewall, -perl-rs} 00/52] Add SDN Fabrics X-BeenThere: pve-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox VE development discussion <pve-devel.lists.proxmox.com> List-Unsubscribe: <https://lists.proxmox.com/cgi-bin/mailman/options/pve-devel>, <mailto:pve-devel-request@lists.proxmox.com?subject=unsubscribe> List-Archive: <http://lists.proxmox.com/pipermail/pve-devel/> List-Post: <mailto:pve-devel@lists.proxmox.com> List-Help: <mailto:pve-devel-request@lists.proxmox.com?subject=help> List-Subscribe: <https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel>, <mailto:pve-devel-request@lists.proxmox.com?subject=subscribe> Reply-To: Proxmox VE development discussion <pve-devel@lists.proxmox.com> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: pve-devel-bounces@lists.proxmox.com Sender: "pve-devel" <pve-devel-bounces@lists.proxmox.com> On 4/3/25 16:20, Friedrich Weber wrote: > On 03/04/2025 16:03, Stefan Hanreich wrote: >> >> >> On 4/3/25 15:44, Friedrich Weber wrote: >>>>> - when removing a fabric, the IP addresses defined on the interfaces >>>>> remain until the next reboot. I guess the reason is that ifupdown2 >>>>> doesn't remove IP addresses when the corresponding stanza vanishes. Not >>>>> sure if this can be easily fixed -- if not, maybe this would be worth a >>>>> note in the docs? >>>> >>>> Umm, I think `ifreload -a` should remove all the addresses? At least it >>>> works on my machine :) >>>> >>>> But I'll check again. >>> >>> I took a closer look -- seems I can only reproduce this if >>> /etc/network/interfaces contains an empty `iface INTERFACE inet manual` >>> stanza for the interface. Without such a stanza, the IP address is >>> removed correctly. >> >> `manual` means, that IP addresses are configured manually by the user, >> so if ifupdown2 encounters an address configured on that interface it >> won't remove it, since you're telling it with manual that it isn't >> responsible for managing addresses on that interface. So I'd say that's >> expected with that line? > > Hmm, the explanation makes sense, but seems like our installer > automatically adds [1] an `iface INTERFACE inet manual` stanza for all > "unused" interfaces? So users may run into this (admittedly minor) issue > if they used interfaces that were already present at installation time > for a fabric, and then remove that fabric. > > [1] > https://git.proxmox.com/?p=pve-installer.git;a=blob;f=Proxmox/Install.pm;h=57fd899;hb=95f2bc3ee#l1097 That's a very valid point - and maybe not even that unlikely in practice. We should either document this, or find a way around it. I'll try to think of a solution, but this one is probably a bit tricky to tackle with how the config generation currently works. _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel