Patch Submission: Bonded interfaces can break Proxmox tools

[Sorthum]

Not sure if there's a better place to submit this; I got no feedback on the user support list.

In my environment we've set up port bonding and in turn disabled vmbr0. This winds up breaking a host of utilities, as $localip doesn't get defined properly in Cluster.pm.

Patching the file as follows resolves this issue neatly, and I'd like to see it included upstream so I don't have to continue patching it in my environment by hand.

- --- Cluster.pm~	2011-04-20 12:20:45.000000000 -0700
+++ Cluster.pm	2011-04-20 12:20:45.000000000 -0700
@@ -179,7 +179,7 @@
    my $hostname = PVE::Config::read_file ("hostname");
    my $ccfg = PVE::Config::read_file ($filename);

- -    my $localip = $ifaces->{vmbr0}->{address} || $ifaces->{eth0}->{address};
+    my $localip = $ifaces->{vmbr0}->{address} || $ifaces->{eth0}->{address} || $ifaces->{bond0}->{address};

    my $cinfo;

 

[dietmar]

That part of the design will be completely new in PVE 2.0, so maybe you can maintain that patch yourself in the meantime?