[pve-devel] [PATCH cluster 2/2] add explaining comment for observed files

Thomas Lamprecht t.lamprecht at proxmox.com
Thu Dec 20 08:53:11 CET 2018


On 12/19/18 11:24 AM, Dominik Csapak wrote:
> Without knowing that those are also defined elsewhere, the code can be
> easily confusing.
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  data/PVE/Cluster.pm | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/data/PVE/Cluster.pm b/data/PVE/Cluster.pm
> index f96de2f..ce4ae7c 100644
> --- a/data/PVE/Cluster.pm
> +++ b/data/PVE/Cluster.pm
> @@ -67,6 +67,9 @@ my $rootsshauthkeys = "/root/.ssh/authorized_keys";
>  my $rootsshauthkeysbackup = "${rootsshauthkeys}.org";
>  my $rootsshconfig = "/root/.ssh/config";
>  
> +# also defined in status.c
> +# this is necessary to be sure that we have a version of that file
> +# when using read/write, so that we can update the cache accordingly
>  my $observed = {
>      'vzdump.cron' => 1,
>      'storage.cfg' => 1,
> 

FYI: I applied a comment  based in my suggestions, but thanks nonetheless,
it's always easier to do something if one has a comparable reference!




More information about the pve-devel mailing list