#573 closed defect (fixed)

qvm-ls -n output: display meanigful networking info

Reported by: joanna Owned by: marmarek
Priority: major Milestone: Release 1
Component: core Keywords:
Cc:

Description

The IP that is displayed as 'gateway' makes no sense. It should rather be displayed as the netvm's IP address (although then it makes problematic to what display in this field for proxy vms). So, it would be better, perhaps to display two IP addresses for each VM:
1) IP (front)
2) IP (back)

Any AppVM will have only IP (front), while any NetVM will have only IP (back -- this would be the 10.137.x.1 pseudo address through which other VMs see this netvm. Proxy VMs, however, will have both of them.

The gateway field of any AppVM/HVM or ProxyVM, should match the IP (back) of the corresponding Proxy/NetVM to which it is connected.

NetVMs should have nothing displayed in the gateway field.

Qubes Manager should be adjust adjusted accordingly.

Change History (5)

comment:1 Changed 12 months ago by marmarek

The IP in 'gateway' field isn't real IP at all (isn't assigned to any interface) - only used to DNAT DNS traffic. It is used as default gateway only to not fill ARP cache (but really there can be used ANY address) - see #355.
Also introducing some "back" and "front" names will only confuse not-so-advanced user.
IMHO the 'gateway' field can be renamed to 'DNS', but no other change.

comment:2 Changed 12 months ago by joanna

That's I wrote 'pseudo' address. Anyway, it isn't important whether it is real or pseudo IP -- what is important is to let the (advanced) user to easily figure out how VM's are interconnected in terms of IP routing. Hence, the idea of using ip front and back. I don't see how this could be confusing -- on the contrary, the current display model is really confusing.

comment:3 Changed 12 months ago by marmarek

Current display model show configuration for VM - eg when the user want to configure it manually (eg when using Windows PV network driver). Still doesn't understand why you want to introduce some IP needed for one VM in row describing other VM.

BTW You propose to add additional column to already wide table (currently about 120 chars), not everyone use fullscreen terminal

comment:4 Changed 12 months ago by marmarek

  • Owner changed from joanna to marmarek
  • Status changed from new to accepted

Finally agreed on replacing 'netmask' column with 'ip back'.

comment:5 Changed 12 months ago by marmarek

  • Resolution set to fixed
  • Status changed from accepted to closed
Note: See TracTickets for help on using tickets.