From: Thomas Lamprecht <t.lamprecht@proxmox.com>
To: pve-devel@lists.proxmox.com, Christoph Heiss <c.heiss@proxmox.com>
Subject: applied: [PATCH installer v5 00/23] fix #2164: add bond and vlan setup across all installers
Date: Sat, 7 Nov 2026 23:07:48 +0100 [thread overview]
Message-ID: <179408895108.3020734.14149092579399794245.b4-ty@b4> (raw)
In-Reply-To: <20261106131058.804870-1-c.heiss@proxmox.com>
On Fri, 06 Nov 2026 14:10:29 +0100, Christoph Heiss wrote:
> Fixes #2164 [0] by adding the possibility to set up a bond and a VLAN tag for
> the management interface during the installation, for all of GUI, TUI and
> auto-installer.
>
> For the GUI and TUI, the current name pinning dialog is converted into a
> full-fledged advanced network options dialog, in the same vein as with
> the bootdisk options.
>
> [...]
Applied, many thanks!
I folded a few changes into your patches, each noted in a [TL: ...]
block of the commit message. The main ones:
- generator: name the VLAN interface 'vlan<ID>' with an explicit raw
device, as '<device>.<ID>' can exceed the interface name length limit
with long predictable names. Resolve bond members to physical links,
the first by name for a MAC address shared by several, and keep the
generated configuration unchanged for setups without a bond or VLAN.
- auto-installer: pass the bond options with the keys and lowercase MAC
addresses the low-level installer expects, reject unknown members and
DHCP leases not received on a member, also set up the bond with the
'from-answer' source, and mark all members as management interfaces
in the post-hook data (schema 1.3).
- GUI and TUI: read and check the options only while bonding or the VLAN
is enabled, take the VLAN tag from a plain entry, as the spin button
silently replaced invalid tags by the nearest valid one, show 802.3ad
as LACP and label members with their pinned names. Patches 22 and 23
are folded into 10 and 18.
- the TUI and auto-installer leave out links without an Ethernet MAC
address, like InfiniBand ones, which the `MacAddress` type cannot
hold.
- structs are adapted in the commits changing them, so that every commit
builds, and the series requires proxmox-installer-types 0.3.1.
On top, as separate commits:
- GUI: keep the advanced options dialog open if it is closed via the
window with invalid input, and modal until it is closed.
- note the balance-alb and balance-tlb modes as unsuitable with a
bridge, in all installers.
- support a tagged network in the installation environment, via the
'proxmox-vlan' kernel parameter or 'prepare-iso --vlan-id', and use
its tag for the management network by default.
- reject pinned names of the form bondN, vlanN and vmbrN.
- select bond members by udev properties in the answer file.
- label bond members with the pinned names as entered in the dialog.
- TUI: avoid an underflow in the tabbed view with narrow tab contents.
[01/23] install: run make tidy
commit: e2644833648f89de89b95345b0858cdc3880eda9
[02/23] tree-wide: use `MacAddress` type instead of string for MAC addresses
commit: 6fe01041152e5ceb27cfcce588a08d0ae18b8c26
[03/23] install: factor out ifupdown2 config generation into own subroutine
commit: ab268db2123c476069ac2014ab8a690ecf136f79
[04/23] install: do not rely on interface name map to be fully populated
commit: 48acddf76734b3d3282d88a85082360eedb0b64c
[05/23] install: config: add option for setting up bond on management interface
commit: 7855417047658554183d941a23e2cf4f50701ade
[06/23] install: config: add option for setting up vlan on management interface
commit: 61c0c6f361ce86468aa24c6afa2a373b780c3662
[07/23] install: network: set up management bond if requested
commit: 0543d1e123d52142929906e025f59651bd9a5f40
[08/23] install: network: set up vlan on management interface if requested
commit: 6fec9d7fd40fa77fef3fa84d8b6a055f30dcf0bf
[09/23] gui: network: factor out name pinning into advanced options dialog
commit: 210ec2c7a6760610c4ea0d38bf8f52bb0db62c74
[10/23] gui: network: add bond setup options to advanced options dialog
commit: 9855583d1be34897ba698739b5e4f767a46296c2
[11/23] gui: network: add vlan setup options to advanced network dialog
commit: 0d38555bffc8eb07ccec38336fa10e0ffe0e983c
[12/23] auto: pass through management network bond options to low-level installer
commit: cf6717a51358b7e8544daf15f2a114e955e326c3
[13/23] auto: pass through management network vlan options to low-level installer
commit: 8c0b5f5ec25de4608e8b2e3961b936a1d15b0f02
[14/23] tui: drop long obsolete comment about logo formatting
commit: 2cd910fc3511a715d362874977eb996c500fe4e4
[15/23] tui: views: add widget for displaying a group of checkboxes
commit: f0ac4c5d9f284fdc8a00abd63b0ce8478e509324
[16/23] tui: network: factor out name pinning into advanced options dialog
commit: 7afba62dcf3e1930504c8168fd97b151c27250ba
[17/23] tui: network: match form label style when enabling/disabling elements
commit: 854425f8e0327b56bc38919b49259fe8e939e4bb
[18/23] tui: network: add bond setup options to advanced network dialog
commit: b492fc69543c70a5fba44aa31dec28f5586eaa0f
[19/23] tui: network: add vlan setup options to advanced network dialog
commit: 065c9ba43454d38d8810d1019230a33c2d5624c5
[20/23] gui: network: split advanced options dialog into tabs
commit: 3e15d0376b44d86f34edd10fd8421b3ab4091731
[21/23] tui: network: split advanced options dialog into tabs
commit: 6d0b559ecbd67d1a6073b2d904bf7ef493a15ecf
[22/23] gui: network: list bond member candidates sharing a MAC address once
(folded into 10/23)
[23/23] tui: network: list bond member candidates sharing a MAC address once
(folded into 18/23)
prev parent reply other threads:[~2026-11-07 22:09 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-11-06 13:10 [PATCH installer v5 00/23] fix #2164: add bond and vlan setup across all installers Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 01/23] install: run make tidy Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 02/23] tree-wide: use `MacAddress` type instead of string for MAC addresses Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 03/23] install: factor out ifupdown2 config generation into own subroutine Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 04/23] install: do not rely on interface name map to be fully populated Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 05/23] install: config: add option for setting up bond on management interface Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 06/23] install: config: add option for setting up vlan " Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 07/23] install: network: set up management bond if requested Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 08/23] install: network: set up vlan on management interface " Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 09/23] gui: network: factor out name pinning into advanced options dialog Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 10/23] gui: network: add bond setup options to " Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 11/23] gui: network: add vlan setup options to advanced network dialog Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 12/23] auto: pass through management network bond options to low-level installer Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 13/23] auto: pass through management network vlan " Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 14/23] tui: drop long obsolete comment about logo formatting Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 15/23] tui: views: add widget for displaying a group of checkboxes Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 16/23] tui: network: factor out name pinning into advanced options dialog Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 17/23] tui: network: match form label style when enabling/disabling elements Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 18/23] tui: network: add bond setup options to advanced network dialog Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 19/23] tui: network: add vlan " Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 20/23] gui: network: split advanced options dialog into tabs Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 21/23] tui: " Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 22/23] gui: network: list bond member candidates sharing a MAC address once Christoph Heiss
2026-11-06 13:10 ` [PATCH installer v5 23/23] tui: " Christoph Heiss
2026-11-07 22:07 ` Thomas Lamprecht [this message]
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=179408895108.3020734.14149092579399794245.b4-ty@b4 \
--to=t.lamprecht@proxmox.com \
--cc=c.heiss@proxmox.com \
--cc=pve-devel@lists.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