mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-12-23 06:05:27 +00:00
interface: dump inactive xml when interface isn't active
Other drivers in libvirt (e.g. network, qemu) will automatically return the "inactive" (persistent configuration) XML of an object when that object is inactive. The netcf backend of the interface driver would always try to return the live status XML of the interface, even when it was down. Although netcf does return valid XML in that case, for bond interfaces it is missing almost all of its content, including the <bond> subelement itself, leading to this error message from "virsh iface-dumpxml" of a bond interface that is inactive: error: XML error: bond interface misses the bond element (this is because libvirt's validation of the XML returned by netcf always requires a <bond> element be present). This patch modifies the interface driver netcf backend to check if the interface is inactive, and in that case always return the inactive XML (which will always have a <bond> element, thus eliminating the error message, as well as making operation more in line with other drivers. This fixes the following bug: https://bugzilla.redhat.com/show_bug.cgi?id=878394
This commit is contained in:
parent
09567144d6
commit
7284c499e5
@ -801,6 +801,7 @@ static char *netcfInterfaceGetXMLDesc(virInterfacePtr ifinfo,
|
||||
char *xmlstr = NULL;
|
||||
virInterfaceDefPtr ifacedef = NULL;
|
||||
char *ret = NULL;
|
||||
bool active;
|
||||
|
||||
virCheckFlags(VIR_INTERFACE_XML_INACTIVE, NULL);
|
||||
|
||||
@ -812,7 +813,10 @@ static char *netcfInterfaceGetXMLDesc(virInterfacePtr ifinfo,
|
||||
goto cleanup;
|
||||
}
|
||||
|
||||
if ((flags & VIR_INTERFACE_XML_INACTIVE)) {
|
||||
if (netcfInterfaceObjIsActive(iface, &active) < 0)
|
||||
goto cleanup;
|
||||
|
||||
if ((flags & VIR_INTERFACE_XML_INACTIVE) || !active) {
|
||||
xmlstr = ncf_if_xml_desc(iface);
|
||||
} else {
|
||||
xmlstr = ncf_if_xml_state(iface);
|
||||
|
Loading…
Reference in New Issue
Block a user