all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [PATCH qemu-server] migration: print netdevice: only warn about missing host_mtu setting when unexpected
@ 2026-03-31 14:46 Fiona Ebner
  2026-03-31 18:41 ` applied: " Thomas Lamprecht
  0 siblings, 1 reply; 2+ messages in thread
From: Fiona Ebner @ 2026-03-31 14:46 UTC (permalink / raw)
  To: pve-devel

When the machine version is less than 10.0+pve1 and the MTU is 1500,
not having the host_mtu parameter is fully expected. Only log when the
missing host_mtu setting is not expected to avoid confusion. Reported
in the community forum [0].

[0]: https://forum.proxmox.com/threads/182156/

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---
 src/PVE/QemuServer.pm | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/PVE/QemuServer.pm b/src/PVE/QemuServer.pm
index fee93193..3e510e01 100644
--- a/src/PVE/QemuServer.pm
+++ b/src/PVE/QemuServer.pm
@@ -1386,7 +1386,12 @@ sub print_netdevice_full {
     } elsif (defined($mtu)) {
         my $msg_prefix = "netdev $netid: ignoring MTU '$mtu'";
         if ($migration_skip_host_mtu) {
-            log_warn("$msg_prefix, not used on the source side according to migration parameters");
+            # When the machine version is less than 10.0+pve1 and the MTU is 1500, not having the
+            # host_mtu parameter is fully expected. Only log when not expected to avoid confusion.
+            if (min_version($machine_version, 10, 0, 1) || $mtu != 1500) {
+                log_warn(
+                    "$msg_prefix, not used on the source side according to migration parameters");
+            }
         } elsif (!$net->{bridge}) {
             log_warn("$msg_prefix, no bridge configured");
         } else {
-- 
2.47.3





^ permalink raw reply	[flat|nested] 2+ messages in thread

* applied: [PATCH qemu-server] migration: print netdevice: only warn about missing host_mtu setting when unexpected
  2026-03-31 14:46 [PATCH qemu-server] migration: print netdevice: only warn about missing host_mtu setting when unexpected Fiona Ebner
@ 2026-03-31 18:41 ` Thomas Lamprecht
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Lamprecht @ 2026-03-31 18:41 UTC (permalink / raw)
  To: pve-devel, Fiona Ebner

On Tue, 31 Mar 2026 16:46:03 +0200, Fiona Ebner wrote:
> When the machine version is less than 10.0+pve1 and the MTU is 1500,
> not having the host_mtu parameter is fully expected. Only log when the
> missing host_mtu setting is not expected to avoid confusion. Reported
> in the community forum [0].
> 
> [0]: https://forum.proxmox.com/threads/182156/
> 
> [...]

Applied, thanks!

[1/1] migration: print netdevice: only warn about missing host_mtu setting when unexpected
      commit: 359712a70837fc1e9854af8c02c504ee7a52e24c




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-03-31 18:40 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-03-31 14:46 [PATCH qemu-server] migration: print netdevice: only warn about missing host_mtu setting when unexpected Fiona Ebner
2026-03-31 18:41 ` applied: " Thomas Lamprecht

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal