[PATCH 0/7] network define: add support for validation against schema

Kristina Hanicova posted 7 patches 2 years, 8 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1629737311.git.khanicov@redhat.com
docs/manpages/virsh.rst           |  4 ++-
include/libvirt/libvirt-network.h |  7 ++++++
src/conf/domain_conf.c            |  2 +-
src/conf/network_conf.c           | 15 ++++++-----
src/conf/network_conf.h           |  3 ++-
src/driver-network.h              |  6 +++++
src/esx/esx_network_driver.c      | 16 ++++++++++--
src/libvirt-network.c             | 41 +++++++++++++++++++++++++++++++
src/libvirt_public.syms           |  1 +
src/network/bridge_driver.c       | 22 +++++++++++++----
src/qemu/qemu_process.c           |  2 +-
src/remote/remote_driver.c        |  1 +
src/remote/remote_protocol.x      | 19 +++++++++++++-
src/remote_protocol-structs       |  8 ++++++
src/test/test_driver.c            | 20 ++++++++++++---
src/vbox/vbox_network.c           | 18 +++++++++++---
tools/virsh-network.c             | 13 +++++++++-
17 files changed, 171 insertions(+), 27 deletions(-)
[PATCH 0/7] network define: add support for validation against schema
Posted by Kristina Hanicova 2 years, 8 months ago
Kristina Hanicova (7):
  api: add public virNetworkDefineXMLFlags() and remote protocol
  vbox_network: add flags to vboxNetworkDefineCreateXML()
  src: add driver support for networkDefineXMLFlags()
  api: add virNetworkDefineFlags
  src & network_conf: add validation against schema in define
  network: allow VIR_NETWORK_DEFINE_VALIDATE flag
  virsh: add support for '--validate' option in define network

 docs/manpages/virsh.rst           |  4 ++-
 include/libvirt/libvirt-network.h |  7 ++++++
 src/conf/domain_conf.c            |  2 +-
 src/conf/network_conf.c           | 15 ++++++-----
 src/conf/network_conf.h           |  3 ++-
 src/driver-network.h              |  6 +++++
 src/esx/esx_network_driver.c      | 16 ++++++++++--
 src/libvirt-network.c             | 41 +++++++++++++++++++++++++++++++
 src/libvirt_public.syms           |  1 +
 src/network/bridge_driver.c       | 22 +++++++++++++----
 src/qemu/qemu_process.c           |  2 +-
 src/remote/remote_driver.c        |  1 +
 src/remote/remote_protocol.x      | 19 +++++++++++++-
 src/remote_protocol-structs       |  8 ++++++
 src/test/test_driver.c            | 20 ++++++++++++---
 src/vbox/vbox_network.c           | 18 +++++++++++---
 tools/virsh-network.c             | 13 +++++++++-
 17 files changed, 171 insertions(+), 27 deletions(-)

-- 
2.31.1

Re: [PATCH 0/7] network define: add support for validation against schema
Posted by Michal Prívozník 2 years, 8 months ago
On 8/23/21 6:50 PM, Kristina Hanicova wrote:
> 
> Kristina Hanicova (7):
>   api: add public virNetworkDefineXMLFlags() and remote protocol
>   vbox_network: add flags to vboxNetworkDefineCreateXML()
>   src: add driver support for networkDefineXMLFlags()
>   api: add virNetworkDefineFlags
>   src & network_conf: add validation against schema in define
>   network: allow VIR_NETWORK_DEFINE_VALIDATE flag
>   virsh: add support for '--validate' option in define network
> 
>  docs/manpages/virsh.rst           |  4 ++-
>  include/libvirt/libvirt-network.h |  7 ++++++
>  src/conf/domain_conf.c            |  2 +-
>  src/conf/network_conf.c           | 15 ++++++-----
>  src/conf/network_conf.h           |  3 ++-
>  src/driver-network.h              |  6 +++++
>  src/esx/esx_network_driver.c      | 16 ++++++++++--
>  src/libvirt-network.c             | 41 +++++++++++++++++++++++++++++++
>  src/libvirt_public.syms           |  1 +
>  src/network/bridge_driver.c       | 22 +++++++++++++----
>  src/qemu/qemu_process.c           |  2 +-
>  src/remote/remote_driver.c        |  1 +
>  src/remote/remote_protocol.x      | 19 +++++++++++++-
>  src/remote_protocol-structs       |  8 ++++++
>  src/test/test_driver.c            | 20 ++++++++++++---
>  src/vbox/vbox_network.c           | 18 +++++++++++---
>  tools/virsh-network.c             | 13 +++++++++-
>  17 files changed, 171 insertions(+), 27 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal