in api listing for all interfaces, and in the GET call for an individual interface.
If we don't show them, the user might be confused if the bridge ports and interface names don't correlate. enables the additional alternative names column in the network view on the gui. Signed-off-by: Dominik Csapak <[email protected]> --- PVE/API2/Network.pm | 32 ++++++++++++++++++++++++++++++-- www/manager6/node/Config.js | 1 + 2 files changed, 31 insertions(+), 2 deletions(-) diff --git a/PVE/API2/Network.pm b/PVE/API2/Network.pm index a8cd7649..19611cb9 100644 --- a/PVE/API2/Network.pm +++ b/PVE/API2/Network.pm @@ -12,6 +12,7 @@ use PVE::RESTHandler; use PVE::RPCEnvironment; use PVE::JSONSchema qw(get_standard_option); use PVE::AccessControl; +use PVE::Network; use IO::File; use base qw(PVE::RESTHandler); @@ -231,6 +232,23 @@ sub json_config_properties { return $prop; } +sub extract_altnames { + my ($iface, $altnames) = @_; + + $altnames = PVE::Network::altname_mapping() if !defined($altnames); + + my $iface_altnames = []; + + for my $altname (keys $altnames->%*) { + if ($altnames->{$altname} eq $iface) { + push $iface_altnames->@*, $altname; + } + } + + return $iface_altnames if scalar($iface_altnames->@*) > 0; + return undef; +} + __PACKAGE__->register_method({ name => 'index', path => '', @@ -390,6 +408,8 @@ __PACKAGE__->register_method({ my $ifaces = $config->{ifaces}; + my $altnames = PVE::Network::altname_mapping(); + delete $ifaces->{lo}; # do not list the loopback device if (my $tfilter = $param->{type}) { @@ -424,6 +444,9 @@ __PACKAGE__->register_method({ my $type = $ifaces->{$k}->{type}; delete $ifaces->{$k} if ($type eq 'bridge' || $type eq 'OVSBridge') && !$can_access_vnet->($k); + + my $iface_altnames = extract_altnames($k, $altnames); + $ifaces->{$k}->{altnames} = $iface_altnames if defined($iface_altnames); } return PVE::RESTHandler::hash_to_array($ifaces, 'iface'); @@ -789,10 +812,15 @@ __PACKAGE__->register_method({ my $config = PVE::INotify::read_file('interfaces'); my $ifaces = $config->{ifaces}; + my $iface = $param->{iface}; + raise_param_exc({ iface => "interface does not exist" }) - if !$ifaces->{ $param->{iface} }; + if !$ifaces->{$iface}; + + my $altnames = extract_altnames($iface); + $ifaces->{$iface}->{altnames} = $altnames if defined($altnames); - return $ifaces->{ $param->{iface} }; + return $ifaces->{$iface}; }, }); diff --git a/www/manager6/node/Config.js b/www/manager6/node/Config.js index 4cd1671c..f6cd8749 100644 --- a/www/manager6/node/Config.js +++ b/www/manager6/node/Config.js @@ -190,6 +190,7 @@ Ext.define('PVE.node.Config', { iconCls: 'fa fa-exchange', itemId: 'network', showApplyBtn: true, + showAltNames: true, groups: ['services'], nodename: nodename, editOptions: { -- 2.39.5 _______________________________________________ pve-devel mailing list [email protected] https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
