[pbs-devel] [PATCH proxmox-backup 00/33] integrate notification system

Gabriel Goller g.goller at proxmox.com
Fri Apr 12 15:59:55 CEST 2024


First of all, thanks for implementing this! I can't count how many times
I've seen a forum post asking for better notifications in pbs!

Some high-level comments:
 - The 'hostname' field doesn't make any sense in pbs (AFAICT) because
   there is only one node.
 - The 'type' field could have hardcoded values, such as 'gc', 'verify',
   etc. this way we can also have a drop-down menu in the ui. This makes
   it a lot easier than checking the docs/guessing.
 - A notification when a backup succeeds/fails would be very useful as
   well.
 - Also having datastore and namespace filters on the notification
   matchers would be great (if feasible obv.).


Another thing I noticed is that creating a datastore through the cli 
still uses the legacy notification system.

Will make a more in depth review next week!





More information about the pbs-devel mailing list