Re: [pve-devel] [PATCH container/manager v4 0/3] Show container ip in summary and network tab

[Daniel Herzig <d.herzig@proxmox.com>]

I just tested this series against an alpine container with sdn simple zone dhcp
IP assignment and it seems to do what it promises.

This makes a very nice streamlined experience with VM behaviour.

Tested-by: Daniel Herzig <d.herzig@proxmox.com>


Gabriel Goller <g.goller@proxmox.com> writes:

> Show the ip/hwaddress of the network interfaces of containers in the summary
> tab of the container and in the network tab on a per-interface basis.
>
> This series was originally by Leo Nunner: 
> https://lore.proxmox.com/pve-devel/20230615094333.66179-1-l.nunner@proxmox.com/
>
> v4, thanks @Daniel:
>  - remove duplicate code (copy paste error)
>  - rebase on latest master
>
> v3, thanks @Thomas, @Maximiliano:
>  - fixed wording in schema description
>  - stay backwards-compatible by keeping old attributes
>  - use array reference instead of array
>
> v2, thanks @Dominik:
>  - show if ip is static or dynamic (dhcp) 
>  - show multiple ips per interface
>  - refactor/reuse AgentIPView instead of adding ContainerIPView component
>  - various other small improvements
>
> manager:
>
> Gabriel Goller (2):
>   lxc: show dynamically assigned IPs in network tab
>   guest: refactor and reuse AgentIPView for containers
>
>  www/manager6/Makefile                         |   2 +-
>  www/manager6/lxc/Network.js                   |  85 +++++++----
>  www/manager6/panel/GuestStatusView.js         | 140 +++++++++++++++++-
>  www/manager6/panel/GuestSummary.js            |   2 +-
>  .../{qemu/AgentIPView.js => panel/IPView.js}  |  79 ++--------
>  5 files changed, 211 insertions(+), 97 deletions(-)
>  rename www/manager6/{qemu/AgentIPView.js => panel/IPView.js} (58%)
>
>
> container:
>
> Gabriel Goller (1):
>   api: return all addresses of an interface
>
>  src/PVE/API2/LXC.pm | 37 +++++++++++++++++++++++++++++++++++--
>  src/PVE/LXC.pm      | 15 +++++++++++++++
>  2 files changed, 50 insertions(+), 2 deletions(-)
>
>
> Summary over all repositories:
>   7 files changed, 261 insertions(+), 99 deletions(-)


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel