[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH v2] Allow 10 chars for domain IDs & 30 chars for names in virsh list



On 01/12/2012 01:19 PM, Daniel P. Berrange wrote:
From: "Daniel P. Berrange"<berrange redhat com>

Domain IDs are at least 16 bits for most hypervisors, theoretically
event 32-bits.  3 characters is clearly too small an alignment.
Increase alignment to 10 characters to allow 32-bit domain IDs to
display cleanly. Commonly seen with LXC where domain IDs are the
process IDs by default.  Also increase the 'name' field from 20
to 30 characters to cope with longer guest names which are quite
common

In v2:

  - Increase size to 10 chars
  - Left align ID values
  - Also increase name field to 30
  - Increase number of '-' to compensate

---
I think that 10 characters is a lot of space wasted in most of cases :/.
What about calculating the alignment dynamicaly? I'm adding code to the list command, so I could add that as a part of my patch.


Peter


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]