- Add new option 'notification-mode'
  Can either be 'mail' or 'channel', selects wheter notifications should
  be sent via email or via a notification channel
- Add new option 'notification-channel'
  Allows to select a channel via which notifications shall be sent (if
  'notification-mode' is set to 'channel')
- Add new option 'notification-policy'
  Replacement for the now deprecated 'mailnotification' option. Mostly
  just a rename for consistency, but also adds the 'never' option.
- Mark 'mailnotification' as deprecated in favor of 'notification-policy'

Signed-off-by: Lukas Wagner <l.wag...@proxmox.com>
---
 src/PVE/VZDump/Common.pm | 28 ++++++++++++++++++++++++++--
 1 file changed, 26 insertions(+), 2 deletions(-)

diff --git a/src/PVE/VZDump/Common.pm b/src/PVE/VZDump/Common.pm
index 4b0e8e0..d3cf8a7 100644
--- a/src/PVE/VZDump/Common.pm
+++ b/src/PVE/VZDump/Common.pm
@@ -164,16 +164,40 @@ my $confdesc = {
        type => 'string',
        format => 'email-or-username-list',
        description => "Comma-separated list of email addresses or users that 
should" .
-           " receive email notifications.",
+           " receive email notifications. Only has an effect if 
'notification-mode' is".
+           " set to 'mail'.",
        optional => 1,
     },
     mailnotification => {
        type => 'string',
-       description => "Specify when to send an email",
+       description => "Deprecated: use 'notification-policy' instead.",
        optional => 1,
        enum => [ 'always', 'failure' ],
        default => 'always',
     },
+    'notification-policy' => {
+       type => 'string',
+       description => "Specify when to send a notification",
+       optional => 1,
+       enum => [ 'always', 'failure', 'never'],
+       default => 'always',
+    },
+    'notification-mode' => {
+       type => 'string',
+       description => "Determine whether to notify via email of via a 
notification channel.",
+       optional => 1,
+       enum => [ 'mail', 'channel' ],
+       default => 'mail',
+    },
+    'notification-channel' => {
+       type => 'string',
+       format => 'pve-configid',
+       description => "Determine the channel via which notifications should be 
sent." .
+           " Only has an effect if 'notification-mode' is set to 'channel'." .
+           " If 'notification-mode' is set to 'channel' and 
'notification-channel' is " .
+           " not set, then no notification will be sent.",
+       optional => 1,
+    },
     tmpdir => {
        type => 'string',
        description => "Store temporary files to specified directory.",
-- 
2.30.2



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to