Re: [PATCH 0/6] Implement XML validation feature for node devices and storage volumes

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

 



On 10/20/22 9:37 AM, Peter Krempa wrote:
In the last round of adding support for built-in validation the node
device APIs and storage volume creation were not covered.

Note that due to the close freeze date I've already marked the APIs for
v8.10.

Peter Krempa (6):
   conf: node_device: Add 'validate' argument to virNodeDeviceDefParse
   nodedev: Add VIR_NODE_DEVICE_(CREATE|DEFINE)_XML_VALIDATE flags
   nodedev|test: Implement support for validating node device XMLs
   conf: storage: Add support for validating storage vol XML to
     virStorageVolDefParse
   storage: Add VIR_STORAGE_VOL_CREATE_VALIDATE flag
   storage|test|vbox: Implement support for validating storage volume
     XMLs

  docs/manpages/virsh.rst              | 19 +++++++++++++++----
  include/libvirt/libvirt-nodedev.h    | 19 +++++++++++++++++++
  include/libvirt/libvirt-storage.h    |  1 +
  src/conf/node_device_conf.c          |  5 +++--
  src/conf/node_device_conf.h          |  3 ++-
  src/conf/storage_conf.c              |  3 ++-
  src/conf/storage_conf.h              |  2 ++
  src/hypervisor/domain_driver.c       |  6 +++---
  src/libvirt-nodedev.c                |  4 ++--
  src/node_device/node_device_driver.c | 10 ++++++----
  src/storage/storage_driver.c         | 18 ++++++++++++++----
  src/test/test_driver.c               | 24 +++++++++++++++++-------
  src/vbox/vbox_storage.c              |  8 ++++++--
  tests/nodedevmdevctltest.c           |  4 ++--
  tests/nodedevxml2xmltest.c           |  3 ++-
  tools/virsh-nodedev.c                | 20 ++++++++++++++++++--
  tools/virsh-volume.c                 | 14 ++++++++++++++
  17 files changed, 128 insertions(+), 35 deletions(-)



Reviewed-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx>




[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