Re: [PATCH 06/16] nwfilter: Introduce public API to retrieve 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:
> Introduces `virNWFilterGetMetadata()` to retrieve user defined
> metadata fields, i.e `<title>`, `<description>` and `<metadata>`
> 
> Signed-off-by: K Shiva Kiran <shiva_kr@xxxxxxxxxx>
> ---
>  include/libvirt/libvirt-nwfilter.h |  5 +++
>  src/driver-nwfilter.h              |  6 +++
>  src/libvirt-nwfilter.c             | 67 ++++++++++++++++++++++++++++++
>  src/libvirt_public.syms            |  1 +
>  4 files changed, 79 insertions(+)

Alright, introducing setter and getter in two different APIs is a bit of
a stretch but I can live with it. Somebody might want just the setter
and use nwfilter-dumpxml as getter.

But then you implement RPC for both of these APIs in a single patch
(07/16). Therefore, please merge also 05 and 06.

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