* [pve-devel] [PATCH cluster] datacenter config: mark notification settings as deprecated/unused
@ 2024-01-19 10:30 Lukas Wagner
2024-03-25 16:56 ` [pve-devel] applied: " Thomas Lamprecht
0 siblings, 1 reply; 2+ messages in thread
From: Lukas Wagner @ 2024-01-19 10:30 UTC (permalink / raw)
To: pve-devel
These were part of the first version of the notification overhaul
which was already rolled out in pvetest. To avoid breakage for users
who may have used the version from pvetest, we do not remove them yet
and only mark them as unused or deprecated. They can be removed at
some point in the future.
Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
---
src/PVE/DataCenterConfig.pm | 47 ++++++++++---------------------------
1 file changed, 12 insertions(+), 35 deletions(-)
diff --git a/src/PVE/DataCenterConfig.pm b/src/PVE/DataCenterConfig.pm
index c80872c..abd0bbf 100644
--- a/src/PVE/DataCenterConfig.pm
+++ b/src/PVE/DataCenterConfig.pm
@@ -52,8 +52,10 @@ my $notification_format = {
'package-updates' => {
type => 'string',
enum => ['auto', 'always', 'never'],
- description => "Control when the daily update job should send out notifications.",
- verbose_description => "Control how often the daily update job should send out notifications:\n"
+ description => "DEPRECATED: Use datacenter notification settings instead."
+ ." Control when the daily update job should send out notifications.",
+ verbose_description => "DEPRECATED: Use datacenter notification settings instead.\n"
+ ."Control how often the daily update job should send out notifications:\n"
."* 'auto' daily for systems with a valid subscription, as those are assumed to be "
." production-ready and thus should know about pending updates.\n"
."* 'always' every update, if there are new pending updates.\n"
@@ -61,62 +63,37 @@ my $notification_format = {
default => 'auto',
optional => 1,
},
+ # TODO: These are the left-overs of the first version of the overhauled notification system.
+ # It was already available on pvetest, thus we should keep these entries here for a while to
+ # avoid any breakage.
'target-package-updates' => {
type => 'string',
format_description => 'TARGET',
- description => "Control where notifications about available updates should be sent to.",
- verbose_description => "Control where notifications about available"
- . " updates should be sent to."
- . " Has to be the name of a notification target (endpoint or notification group)."
- . " If the 'target-package-updates' parameter is not set, the system will send mails"
- . " to root via a 'sendmail' notification endpoint.",
+ description => "UNUSED - Use datacenter notification settings instead.",
optional => 1,
},
'fencing' => {
type => 'string',
enum => ['always', 'never'],
- description => "Control if notifications about node fencing should be sent.",
- verbose_description => "Control if notifications about node fencing should be sent.\n"
- . "* 'always' always send out notifications\n"
- . "* 'never' never send out notifications.\n"
- . "For production systems, turning off node fencing notifications is not"
- . "recommended!\n",
- default => 'always',
+ description => "UNUSED - Use datacenter notification settings instead.",
optional => 1,
},
'target-fencing' => {
type => 'string',
format_description => 'TARGET',
- description => "Control where notifications about fenced cluster nodes should be sent to.",
- verbose_description => "Control where notifications about fenced cluster nodes"
- . " should be sent to."
- . " Has to be the name of a notification target (endpoint or notification group)."
- . " If the 'target-fencing' parameter is not set, the system will send mails"
- . " to root via a 'sendmail' notification endpoint.",
+ description => "UNUSED - Use datacenter notification settings instead.",
optional => 1,
},
'replication' => {
type => 'string',
enum => ['always', 'never'],
- description => "Control if notifications for replication failures should be sent.",
- verbose_description => "Control if notifications for replication failures should be sent.\n"
- . "* 'always' always send out notifications\n"
- . "* 'never' never send out notifications.\n"
- . "For production systems, turning off replication notifications is not"
- . "recommended!\n",
- default => 'always',
+ description => "UNUSED - Use datacenter notification settings instead.",
optional => 1,
},
'target-replication' => {
type => 'string',
format_description => 'TARGET',
- description => "Control where notifications for failed storage replication jobs should"
- . " be sent to.",
- verbose_description => "Control where notifications for failed storage replication jobs"
- . " should be sent to."
- . " Has to be the name of a notification target (endpoint or notification group)."
- . " If the 'target-replication' parameter is not set, the system will send mails"
- . " to root via a 'sendmail' notification endpoint.",
+ description => "UNUSED - Use datacenter notification settings instead.",
optional => 1,
},
};
--
2.39.2
^ permalink raw reply [flat|nested] 2+ messages in thread
* [pve-devel] applied: [PATCH cluster] datacenter config: mark notification settings as deprecated/unused
2024-01-19 10:30 [pve-devel] [PATCH cluster] datacenter config: mark notification settings as deprecated/unused Lukas Wagner
@ 2024-03-25 16:56 ` Thomas Lamprecht
0 siblings, 0 replies; 2+ messages in thread
From: Thomas Lamprecht @ 2024-03-25 16:56 UTC (permalink / raw)
To: Proxmox VE development discussion, Lukas Wagner
Am 19/01/2024 um 11:30 schrieb Lukas Wagner:
> These were part of the first version of the notification overhaul
> which was already rolled out in pvetest. To avoid breakage for users
> who may have used the version from pvetest, we do not remove them yet
> and only mark them as unused or deprecated. They can be removed at
> some point in the future.
>
> Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
> ---
> src/PVE/DataCenterConfig.pm | 47 ++++++++++---------------------------
> 1 file changed, 12 insertions(+), 35 deletions(-)
>
>
applied, thanks!
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-03-25 16:56 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-19 10:30 [pve-devel] [PATCH cluster] datacenter config: mark notification settings as deprecated/unused Lukas Wagner
2024-03-25 16:56 ` [pve-devel] applied: " Thomas Lamprecht
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox