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