Re: [PATCH pve-guest-common 1/3] fix #5713: vzdump: add notification-mode none

[David Riley <d.riley@proxmox.com>]

Small typo in the description.

On 5/13/26 9:26 AM, Erik Fastermann wrote:
> Optionally disable sending notification mails with the new option
> none for notification-mode in vzdump.
>
> This can be useful if you want to disable sending notifications for a
> specific backup job and not change the global notification settings or
> use the deprecated legacy sendmail with an empty address.
>
> Signed-off-by: Erik Fastermann <e.fastermann@proxmox.com>
> ---
>   src/PVE/VZDump/Common.pm | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/src/PVE/VZDump/Common.pm b/src/PVE/VZDump/Common.pm
> index 67a4cf4..1c636b4 100644
> --- a/src/PVE/VZDump/Common.pm
> +++ b/src/PVE/VZDump/Common.pm
> @@ -261,9 +261,10 @@ my $confdesc = {
>               . " specified address(es) via the 'sendmail' command. If set to 'notification-system',"
>               . " a notification will be sent via PVE's notification system, and the mailto and"
>               . " mailnotification will be ignored. If set to 'auto' (default setting), an email will"
> -            . " be sent if mailto is set, and the notification system will be used if not.",
> +            . " be sent if mailto is set, and the notification system will be used if not."
> +            . " If set to 'none', no notification will be send.",
should be "sent" instead of "send".
>           optional => 1,
> -        enum => ['auto', 'legacy-sendmail', 'notification-system'],
> +        enum => ['auto', 'legacy-sendmail', 'notification-system', 'none'],
>           default => 'auto',
>       },
>       tmpdir => {