Re: [PATCH 1/4] virsh: Document limitation of domifstat

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

 



On 10/13/2017 10:44 AM, John Ferlan wrote:
> 
> 
> On 10/05/2017 10:18 AM, Michal Privoznik wrote:
>> https://bugzilla.redhat.com/show_bug.cgi?id=1497396
>>
>> The current implementation reads the stats from the host.
>> However, this doesn't work for all types of interfaces as not all
>> of them have a representation in the host. For instance,
>> interface type='user' doesn't.
>>
>> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
>> ---
>>  tools/virsh.pod | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/tools/virsh.pod b/tools/virsh.pod
>> index 00d93781a..632f202e8 100644
>> --- a/tools/virsh.pod
>> +++ b/tools/virsh.pod
>> @@ -775,7 +775,9 @@ the guest OS via an agent. If unspecified, 'lease' is the default.
>>  
>>  =item B<domifstat> I<domain> I<interface-device>
>>  
>> -Get network interface stats for a running domain.
>> +Get network interface stats for a running domain. This might be
>> +unavailable for some types of interface which don't have
>> +representation in the host, e.g. user.
> 
> What's not really clear is what "don't have representation in the host"
> actually means.  I really think this needs to be a bit more specific as
> to "why" something will fail.
> 
> If being too specific is an issue, then consider instead:
> 
> The network interface stats are only available for interfaces that have
> a physical source interface. This does not include, for example, a
> 'user' interface type since it is a virtual LAN with NAT to the outside
> world.

Okay. I'm going with this version.

> 
> Curious, are there any others that might qualify?

I don't know any other from the top of my head.

> 
> Reviewed-by: John Ferlan <jferlan@xxxxxxxxxx>

Thanks.

Michal

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux