Fix VM and IP alignment in tenant list output

[alessandrocornacchia]

Summary

Fixes #2 - Corrects the misalignment of VM names and their corresponding IP addresses in the p4tenant list command output.

Problem

When displaying tenants with multiple VMs (e.g., user mspina with restvm-mspina-01 and restvm-mspina-02), the IP addresses were not correctly aligned with their VM names. All IPs from all VMs were displayed in sequence without grouping, causing visual misalignment where the second VM name would appear next to an IP address from the first VM.

Before:

restvm-mspina-01    10.10.0.21/24
restvm-mspina-02    10.10.0.22/24
                    10.10.0.11/24
                    10.10.0.12/24

Solution

Modified the tenant information structure and table rendering logic to:

1. In tenant.py: Added vm_ip_map dictionary to get_tenant_info() that maps each VM name to its list of IP addresses, preserving the association between VMs and their IPs.

2. In ui.py: Updated create_tenant_table() to use the vm_ip_map for proper alignment:

   • For tenants with multiple VMs, iterate through each VM
   • Display the VM name with its first IP on the same line
   • Show remaining IPs for that VM on subsequent lines with blank VM names
   • Then move to the next VM

After:

restvm-mspina-01    10.10.0.21/24
                    10.10.0.22/24
restvm-mspina-02    10.10.0.11/24
                    10.10.0.12/24

Testing

Tested with the current configuration where user mspina has two VMs, each with 2 IP addresses. The alignment now correctly groups each VM with its corresponding IPs.

Files Changed

• p4tenant/src/p4tenant/tenant.py: Added vm_ip_map to tenant info structure
• p4tenant/src/p4tenant/ui.py: Implemented proper alignment logic in table rendering

🤖 Generated with Claude Code

[claude]

---

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

---