[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