[PATCH 0/2] Support hooks placed in several files

Michal Privoznik mprivozn at redhat.com
Mon Jun 22 13:06:34 UTC 2020


On 6/21/20 11:16 PM, Dmitry Nesterenko wrote:
> Libvirt hook calls only one script (/etc/libvirt/hooks/<driver>) now.
> This is not convenient if scripts for hook are provided by many vendors.
> Script one vendor can replace previously installed script other vendor.
> These patches are changing behaviour of hooks to usual linux scheme -
> running all scripts from directory /etc/libvirt/hooks/<driver>.d in
> alphabetical order. If we find script in old place we will execute it
> as first for backward compatibility.
> 
> Dmitry Nesterenko (2):
>    virthook: refactoring for support hooks placed in several files
>    virhook: support hooks placed in several files
> 
>   src/util/virhook.c | 168 +++++++++++++++++++++++++++++++++++++++------
>   1 file changed, 146 insertions(+), 22 deletions(-)
> 

What I am missing here is:
- documentation (we have docs/hooks.html.in documenting how the hook 
scripts are handled)
- news.xml - this is user visible change that is possibly interesting to 
them


Otherwise looking good.

Michal




More information about the libvir-list mailing list