[libvirt] [PATCH] Support reporting live interface IP/netmask.
Paolo Bonzini
bonzini at gnu.org
Wed Sep 30 06:54:21 UTC 2009
>>> This patch adds the flag VIR_INTERFACE_XML_INACTIVE to
>>> virInterfaceGetXMLDesc's flags.
>>
>> What about calling it instead VIR_INTERFACE_XML_CONFIG and adding a
>> dummy (0) VIR_INTERFACE_XML_DEVICE, and corresponding --config/--device
>> flags to the virsh command? This would match the source attribute.
>>
>> I'll let others override me happily, though.
>
> Well it's to keep coherent naming with the APIs for other kind of
> objects, for example there is an VIR_DOMAIN_XML_INACTIVE in libvirt.h
> to dump the inactive domain informations (when a domain is
> running/active). Similary 'virsh dumpxml --inactive' is about dumping
> the XML for the domain as it was defined.
Then I'm "happily overridden". :-)
> W.r.t. providing accessors for each informations we provide as part of
> an xml dump in virsh, I could try to add something generic, for example
> as a post processing step based on XPath if people really feel disarmed
> by grabbing something from XML output. But was that really the sense
> of your query ?
No, "matching the source attribute" was just my rationale for naming the
options --config/--device (and the corresponding enum names).
Paolo
More information about the libvir-list
mailing list