Re: [pve-devel] [PATCH manager] Revert "ui: dc: remove notify key from datacenter option view"

[Thomas Lamprecht <t.lamprecht@proxmox.com>]

Am 09/02/2024 um 11:16 schrieb Lukas Wagner:
> This reverts commit c81bca2d28744616098448b81fa58e133d3ac5ed.
> 
> In the first iteration of the notification system, notification
> routing and enabling/disabling notifications was configured via
> the (extended) `notify` parameter in `datacenter.cfg`.
> Because of that, the configuration UI for this parameter was moved to
> a new panel as a part of the notification UI.
> When changing to the newer approach for notification routing (matcher
> based), the "new" panel this setting was moved to was dropped from the
> UI.
> 
> Notification sending for package updates is still influenced by this
> parameter (see bin/pveupdate, line 55), so there should be a way to
> configure this from the GUI. At some point, the `notify` parameter
> should be dropped, but that'd be a thing for a major release.
> 
> Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
> ---
> https://forum.proxmox.com/threads/package-update-notifs-not-working.141182/
> 
> Notes:
>     Alternatively, we could just *always* send package update
>     notifications and just ignore that parameter from now on but this
>     might leave users wondering who have previously set
>     `package-updates=never`...

I'd propose two changes:

- add a hint to redirect users to the new mechanisms so that a future
  deprecation would be more expected (if we already plan that now)

- only show it if defined? While that's a bit magic, it'd avoid that
  users set it, but rather use the new mechanism.
  If, I'd never delete the setting via the UI, so that it doesn't
  suddenly disappears if one switches it from some value to default.

What do you think?