[pmg-devel] [PATCH pmg-api] api: relaod ruledb on updating objecgroups

[pmg-devel] [PATCH pmg-api] api: relaod ruledb on updating objecgroups

[Dominik Csapak]

previously they only contained metadata (name/comment) so for the
rulesystem that was not relevant. Now with the attributes and/invert
we want to reload the ruledb on update too.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 src/PMG/API2/ObjectGroupHelpers.pm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/PMG/API2/ObjectGroupHelpers.pm b/src/PMG/API2/ObjectGroupHelpers.pm
index 3060157..fabfa6a 100644
--- a/src/PMG/API2/ObjectGroupHelpers.pm
+++ b/src/PMG/API2/ObjectGroupHelpers.pm
@@ -289,6 +289,8 @@ sub register_object_group_config_api {
 
 	    $rdb->save_group($og);
 
+	    PMG::DBTools::reload_ruledb();
+
 	    return undef;
 	}});
 }
-- 
2.30.2

Re: [pmg-devel] [PATCH pmg-api] api: relaod ruledb on updating objecgroups

[Dominik Csapak]

argh... sorry for the typo in the subject... i can send a v2 or it can 
be fixed up when applying...

[pmg-devel] applied: [PATCH pmg-api] api: relaod ruledb on updating objecgroups

[Stoiko Ivanov]

Thanks for finding and fixing this!
applied the patch, with the 2 typos in the subject line fixed up,
after testing it (watching the journal while sending mails before/after,
looking for 
`pmg-smtp-filter[333766]: reloading configuration Proxmox_ruledb` )


On Thu, 22 Feb 2024 11:04:57 +0100
Dominik Csapak <d.csapak@proxmox.com> wrote:

> previously they only contained metadata (name/comment) so for the
> rulesystem that was not relevant. Now with the attributes and/invert
> we want to reload the ruledb on update too.
> 
> Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
> ---
>  src/PMG/API2/ObjectGroupHelpers.pm | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/src/PMG/API2/ObjectGroupHelpers.pm b/src/PMG/API2/ObjectGroupHelpers.pm
> index 3060157..fabfa6a 100644
> --- a/src/PMG/API2/ObjectGroupHelpers.pm
> +++ b/src/PMG/API2/ObjectGroupHelpers.pm
> @@ -289,6 +289,8 @@ sub register_object_group_config_api {
>  
>  	    $rdb->save_group($og);
>  
> +	    PMG::DBTools::reload_ruledb();
> +
>  	    return undef;
>  	}});
>  }