Re: [PATCH 11/11] domain: conf: Drop expectedVirtTypes

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

 



On 18.04.2015 03:45, Cole Robinson wrote:
> This needs to specified in way too many places for a simple validation
> check. The ostype/arch/virttype validation checks later in
> DomainDefParseXML should catch most of the cases that this was covering.
> ---
>  src/bhyve/bhyve_driver.c          |  5 ---
>  src/conf/domain_conf.c            | 79 +++++----------------------------------
>  src/conf/domain_conf.h            |  6 ---
>  src/conf/snapshot_conf.c          | 14 ++-----
>  src/conf/snapshot_conf.h          |  2 -
>  src/esx/esx_driver.c              |  4 +-
>  src/libxl/libxl_domain.c          |  1 -
>  src/libxl/libxl_driver.c          |  6 ---
>  src/libxl/libxl_migration.c       |  2 -
>  src/lxc/lxc_controller.c          |  1 -
>  src/lxc/lxc_driver.c              |  5 ---
>  src/openvz/openvz_driver.c        |  3 --
>  src/parallels/parallels_driver.c  |  1 -
>  src/phyp/phyp_driver.c            |  1 -
>  src/qemu/qemu_domain.c            |  1 -
>  src/qemu/qemu_domain.h            |  6 ---
>  src/qemu/qemu_driver.c            | 12 ------
>  src/qemu/qemu_migration.c         |  5 +--
>  src/test/test_driver.c            |  7 ----
>  src/uml/uml_driver.c              |  4 --
>  src/vbox/vbox_common.c            |  4 +-
>  src/vmware/vmware_driver.c        |  2 -
>  src/xen/xen_driver.c              |  3 --
>  tests/domainconftest.c            |  3 +-
>  tests/domainsnapshotxml2xmltest.c |  1 -
>  tests/lxcxml2xmltest.c            |  1 -
>  tests/qemuagenttest.c             |  1 -
>  tests/qemuhotplugtest.c           |  1 -
>  tests/qemuxml2argvtest.c          |  1 -
>  tests/qemuxml2xmltest.c           |  3 +-
>  tests/qemuxmlnstest.c             |  1 -
>  tests/securityselinuxlabeltest.c  |  4 +-
>  tests/xlconfigtest.c              |  1 -
>  tests/xmconfigtest.c              |  1 -
>  tests/xml2sexprtest.c             |  1 -
>  tests/xml2vmxtest.c               |  1 -
>  36 files changed, 19 insertions(+), 175 deletions(-)

This is missing:
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 63b65b9..479b4c2 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -36,7 +36,6 @@
 #include "domain_conf.h"
 #include "snapshot_conf.h"
 #include "viralloc.h"
-#include "verify.h"
 #include "virxml.h"
 #include "viruuid.h"
 #include "virbuffer.h"
diff --git a/src/security/virt-aa-helper.c b/src/security/virt-aa-helper.c
index 37867cf..35423b5 100644
--- a/src/security/virt-aa-helper.c
+++ b/src/security/virt-aa-helper.c
@@ -745,7 +745,6 @@ get_definition(vahControl * ctl, const char *xmlStr)
 
     ctl->def = virDomainDefParseString(xmlStr,
                                        ctl->caps, ctl->xmlopt,
-                                       -1,
                                        VIR_DOMAIN_DEF_PARSE_INACTIVE);
     if (ctl->def == NULL) {
         vah_error(ctl, 0, _("could not parse XML"));
diff --git a/src/xenapi/xenapi_driver.c b/src/xenapi/xenapi_driver.c
index 881f5b2..11f6e91 100644
--- a/src/xenapi/xenapi_driver.c
+++ b/src/xenapi/xenapi_driver.c
@@ -565,7 +565,6 @@ xenapiDomainCreateXML(virConnectPtr conn,
 
     virDomainDefPtr defPtr = virDomainDefParseString(xmlDesc,
                                                      priv->caps, priv->xmlopt,
-                                                     1 << VIR_DOMAIN_VIRT_XEN,
                                                      parse_flags);
     if (!defPtr)
         return NULL;
@@ -1740,7 +1739,6 @@ xenapiDomainDefineXMLFlags(virConnectPtr conn, const char *xml, unsigned int fla
         return NULL;
     virDomainDefPtr defPtr = virDomainDefParseString(xml,
                                                      priv->caps, priv->xmlopt,
-                                                     1 << VIR_DOMAIN_VIRT_XEN,
                                                      parse_flags);
     if (!defPtr)
         return NULL;


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]