[pve-devel] [PATCH pve-cluster] add 'sdn.cfg' && 'sdn.cfg.new' to observed files

[Alexandre DERUMIER]

>>FYI: if this really needs to be an observed file one also needs to add
>>it to the respective hash in data/src/status.c (memdb_change_array)

didn't known that. As almost all datacenter config files where in observed file,
I simply put also sdn.cfg too.

I'm not sure if we need it here ? (Don't know exactly how is work in background)



----- Mail original -----
De: "Thomas Lamprecht" <t.lamprecht at proxmox.com>
À: "pve-devel" <pve-devel at pve.proxmox.com>, "aderumier" <aderumier at odiso.com>
Envoyé: Lundi 17 Juin 2019 11:35:56
Objet: Re: [pve-devel] [PATCH pve-cluster] add 'sdn.cfg' && 'sdn.cfg.new' to observed files

On 6/17/19 11:26 AM, Alexandre Derumier wrote: 
> Signed-off-by: Alexandre Derumier <aderumier at odiso.com> 
> --- 
> data/PVE/Cluster.pm | 2 ++ 
> 1 file changed, 2 insertions(+) 
> 
> diff --git a/data/PVE/Cluster.pm b/data/PVE/Cluster.pm 
> index 18bd235..b53dcd7 100644 
> --- a/data/PVE/Cluster.pm 
> +++ b/data/PVE/Cluster.pm 
> @@ -95,6 +95,8 @@ my $observed = { 
> 'ha/fence.cfg' => 1, 
> 'status.cfg' => 1, 
> 'ceph.conf' => 1, 
> + 'sdn.cfg' => 1, 
> + 'sdn.cfg.new' => 1, 

FYI: if this really needs to be an observed file one also needs to add 
it to the respective hash in data/src/status.c (memdb_change_array) 


> }; 
> 
> # only write output if something fails 
>