[pve-devel] applied: [PATCH pve-common] INotify: remove allow-hotplug from /etc/network/interfaces

[Thomas Lamprecht]

On 1/7/20 1:59 AM, Alexandre Derumier wrote:
> for user installing proxmox on top of debian,
> debian install by default the first nic with allow-hotplug.
> 
> This is conflicting with "auto ...", but worst with ovs "allow-ovs ...".
> 
> User have reported race with ovs, where ovs vmbr was up before the nic.
> https://forum.proxmox.com/threads/no-network-on-server-unless-i-ifdown-ifup-vmbr0.62733/
> 
> Signed-off-by: Alexandre Derumier <aderumier at odiso.com>
> ---
>  src/PVE/INotify.pm | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/PVE/INotify.pm b/src/PVE/INotify.pm
> index 22f413b..867da30 100644
> --- a/src/PVE/INotify.pm
> +++ b/src/PVE/INotify.pm
> @@ -901,6 +901,7 @@ sub __read_etc_network_interfaces {
>      SECTION: while (defined ($line = <$fh>)) {
>  	chomp ($line);
>  	next if $line =~ m/^\s*#/;
> +	next if $line =~ m/^\s*(allow-hotplug)\s+(.*)$/;
>  
>  	if ($line =~ m/^\s*(auto|allow-ovs)\s+(.*)$/) {
>  	    my @aa = split (/\s+/, $2);
> 

applied, nice catch!