Re: [PATCH 00/16] nwfilter: Add support for user defined metadata

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

 



On 9/3/23 17:49, K Shiva Kiran wrote:
> This patchset adds support for the following user defined metadata
> fields for network filters.
> 
> - title: A short description of the filter.
> - description: Any documentation that the user wants to store.
> - metadata: Other metadata in XML form.
> 
> K Shiva Kiran (16):
>   xml: Add <title>, <description> and <metadata> to nwfilter xml schema
>   conf: Add parser logic for nwfilter metadata fields
>   nwfilter: Add enum to operate on user defined metadata
>   nwfilter: Add error code and message for missing metadata
>   nwfilter: Introduce public API to modify user metadata
>   nwfilter: Introduce public API to retrieve user-defined metadata
>   nwfilter: Implement RPC
>   virsh: Add helper method to retrieve xml from NWFilter def
>   virsh: Add new command `nwfilter-desc`
>   virsh: Add new command `nwfilter-metadata`
>   virsh: Add option --title for nwfilter-list
>   docs: Document nwfilter metadata related commands
>   virnwfilterobj: Add virNWFilterObjGetMetadata()
>   virnwfilterobj: Add virNWFilterObjSetMetadata()
>   nwfilter_driver: Add Driver implementation for metadata
>   NEWS: Introduce user-defined metadata fields for NWFilter object
> 
>  NEWS.rst                           |  18 ++
>  docs/formatnwfilter.rst            |  31 +++
>  docs/manpages/virsh.rst            |  98 +++++++-
>  include/libvirt/libvirt-nwfilter.h |  27 ++
>  include/libvirt/virterror.h        |   1 +
>  src/conf/nwfilter_conf.c           |  30 +++
>  src/conf/nwfilter_conf.h           |   5 +
>  src/conf/schemas/nwfilter.rng      |   9 +
>  src/conf/virnwfilterobj.c          | 148 +++++++++++
>  src/conf/virnwfilterobj.h          |  13 +
>  src/driver-nwfilter.h              |  15 ++
>  src/libvirt-nwfilter.c             | 154 ++++++++++++
>  src/libvirt_private.syms           |   2 +
>  src/libvirt_public.syms            |   6 +
>  src/nwfilter/nwfilter_driver.c     |  61 +++++
>  src/remote/remote_driver.c         |   2 +
>  src/remote/remote_protocol.x       |  34 ++-
>  src/remote_protocol-structs        |  19 ++
>  src/util/virerror.c                |   3 +
>  tools/virsh-nwfilter.c             | 387 ++++++++++++++++++++++++++++-
>  tools/virsh-util.c                 |  25 ++
>  tools/virsh-util.h                 |   9 +
>  22 files changed, 1089 insertions(+), 8 deletions(-)
> 

The code looks okay. But the split into patches is a bit awkward. Also,
since I've merged your other series, there's a conflict in RPC
definition file.

Looking forward to v2.

Michal




[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