[PATCH v2 0/3] Support hooks placed in several files

Michal Privoznik mprivozn at redhat.com
Tue Jun 23 16:38:37 UTC 2020


On 6/23/20 4:45 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 the directories /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 (3):
>    virhook: refactoring for support hooks placed in several files
>    virhook: support hooks placed in several files
>    virhook: changes in docs about support hooks placed in several files
> 
>   NEWS.rst           |   6 ++
>   docs/hooks.html.in |  27 ++++++++
>   src/util/virhook.c | 164 +++++++++++++++++++++++++++++++++++++++------
>   3 files changed, 175 insertions(+), 22 deletions(-)
> 

I'm fixing all the small nits, and pushing.

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

Congratulations on your first libvirt contribution!

Michal




More information about the libvir-list mailing list