[pve-devel] [PATCH common] support for predictable network interface device names

[Igor Vlasenko]

thanks! It looks good.

On Tue, May 17, 2016 at 10:21 AM, Wolfgang Bumiller
<w.bumiller at proxmox.com> wrote:
> Based on patch from: Igor Vlasenko <viy at altlinux.org>
> ---
> Since it was just a diff pasted in a response, here the patch in a
> git-am applyable form.
>
>  src/PVE/INotify.pm | 8 ++++----
>  src/PVE/Network.pm | 2 +-
>  2 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/src/PVE/INotify.pm b/src/PVE/INotify.pm
> index 74a0fe1..c34659f 100644
> --- a/src/PVE/INotify.pm
> +++ b/src/PVE/INotify.pm
> @@ -800,7 +800,7 @@ sub __read_etc_network_interfaces {
>
>      if ($proc_net_dev) {
>         while (defined ($line = <$proc_net_dev>)) {
> -           if ($line =~ m/^\s*(eth\d+):.*/) {
> +           if ($line =~ m/^\s*(eth\d+|en[^:.]+):.*/) {
>                 $ifaces->{$1}->{exists} = 1;
>             }
>         }
> @@ -973,7 +973,7 @@ sub __read_etc_network_interfaces {
>                 $ifaces->{$1}->{exists} = 0;
>                 $d->{exists} = 0;
>             }
> -       } elsif ($iface =~ m/^eth\d+$/) {
> +       } elsif ($iface =~ m/^(?:eth\d+|en[^:.]+)$/) {
>             if (!$d->{ovs_type}) {
>                 $d->{type} = 'eth';
>             } elsif ($d->{ovs_type} eq 'OVSPort') {
> @@ -1200,7 +1200,7 @@ sub __write_etc_network_interfaces {
>             $d->{type} eq 'OVSBond') {
>             my $brname = $used_ports->{$iface};
>             if (!$brname || !$ifaces->{$brname}) {
> -               if ($iface =~ /^eth/) {
> +               if ($iface =~ /^(?:eth|en)/) {
>                     $ifaces->{$iface} = { type => 'eth',
>                                           exists => 1,
>                                           method => 'manual',
> @@ -1289,7 +1289,7 @@ NETWORKDOC
>         my $pri;
>         if ($iface eq 'lo') {
>             $pri = $if_type_hash->{loopback};
> -       } elsif ($iface =~ m/^eth\d+$/) {
> +       } elsif ($iface =~ m/^(?:eth\d+|en[^:.]+)$/) {
>             $pri = $if_type_hash->{eth} + $child;
>         } elsif ($iface =~ m/^bond\d+$/) {
>             $pri = $if_type_hash->{bond} + $child;
> diff --git a/src/PVE/Network.pm b/src/PVE/Network.pm
> index be26132..0a984ad 100644
> --- a/src/PVE/Network.pm
> +++ b/src/PVE/Network.pm
> @@ -440,7 +440,7 @@ sub activate_bridge_vlan {
>
>      my @ifaces = ();
>      my $dir = "/sys/class/net/$bridge/brif";
> -    PVE::Tools::dir_glob_foreach($dir, '((eth|bond)\d+(\.\d+)?)', sub {
> +    PVE::Tools::dir_glob_foreach($dir, '(((eth|bond)\d+|en[^.]+)(\.\d+)?)', sub {
>          push @ifaces, $_[0];
>      });
>
> --
> 2.1.4
>
>



-- 
С уважением,
Игорь Власенко.