Needs to be done for the TUI installer as well. Easiest way here is to
add `state` to each interface in Proxmox::Sys::RunEnv::query_netdevs(),
then deserialize that in `setup::Interface`.
Looking at it, adding the MAC address there too would be great, if you
are already at it.
On Fri, Aug 04, 2023 at 12:26:46PM +0200, Filip Schauer wrote:
>
> Signed-off-by: Filip Schauer <f.scha...@proxmox.com>
> ---
> proxinstall | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/proxinstall b/proxinstall
> index d5b2565..9316578 100755
> --- a/proxinstall
> +++ b/proxinstall
> @@ -347,7 +347,7 @@ sub create_ipconf_view {
>
> my $get_device_desc = sub {
> my $iface = shift;
> - return "$iface->{name} - $iface->{mac} ($iface->{driver})";
> + return "$iface->{name} - $iface->{mac} ($iface->{driver}) -
> $iface->{state}";
I think it would be better to only show the state if the interface is
actually UP - thus drawing immediate attention to these list entries and
not completely overloading that dropdown.
Maybe even append it as "(UP)" instead of by dash (as it does not really
identify the interface, but rather is a property of it).
> };
>
> my $run_env = Proxmox::Install::RunEnv::get();
> --
> 2.39.2
>
>
>
> _______________________________________________
> pve-devel mailing list
> pve-devel@lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
>
>
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
