On Wed, 23 Jul 2025 16:21:06 +0200, Stefan Hanreich wrote:
> The function reading /etc/network/interfaces used /proc/net/dev to
> determine pre-existing physical interfaces. Since the introduction of
> altnames, /proc/net/dev returns insufficient information for
> determining if an interface is already contained in /e/n/i, since it
> does not include altnames.
> 
> The interfaces parser added all interfaces from /proc/net/dev as
> configurable network devices. If altnames were used in the
> configuration, then the same interface would be listed twice: once
> with its 'real' name (from /proc/net/dev) and once with its altname
> (from the interfaces file).
> 
> [...]

I tested this one some configurations where /e/n/i used different names
compared to the "primary" interface name the kernels uses, made it work fine
now. Implementation wise it looks relative straight forward.

I do not recall for sure anymore, but do differing bridge-ports work
transparently with the ifupdown2 changes from Christoph. With that it might be
nice to support it here too in the midterm, but that is certainly not a blocker
for now.

Applied with perltidy formatting changes squashed in, thanks!

[1/1] inotify/interfaces: use 'ip link' instead of /proc/net/dev
      commit: 346ff9744b036c529373cab5b503eaaf37a5bf03


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

Reply via email to