[pve-devel] [PATCH manager v2 1/2] gui: lxc/Network: add a virtual 'none' option

[Thomas Lamprecht]

On 2/5/20 12:00 PM, Dominik Csapak wrote:
> sometimes, if users do not want a ipv4/6 address, they set the network mode
> to 'dhcp' instead of 'static' (with no ip set), in believe
> this will result in no ip
> 
> in reality, often no ipv6 dhcp server exists in the environment, and
> the container start stalls for ~5min trying to get a ipv6
> 
> to improve ux, add the option 'none', which functionally is the same
> as having selected 'static' with no ip set, and simultaniously
> make the ip required
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
> changes from v1:
> * make 'static' default for new nics (e.g. in the wizard)
> * rename none4/6 to noIPv4/6Addr
> * remove unnecessary comments
> * use NoneText (this requires a new widget-toolkit with my other patch
>   applied)

So, was about to applied it and tried it a bit out, which got me thinking a bit.
I'm not really sure anymore if this is the best approach.

Why don't we just use a sane emptyText for the "none" case.
This avoids two additional radio buttons (cluttering) while giving a user the
same feedback. Maybe, "No Address"?

What do you think?