[pve-devel] [PATCH docs v6 15/16] notifications: describe new notification metadata fields

[Lukas Wagner]

On  2024-04-22 11:10, Fiona Ebner wrote:
>> +| Cluster node fenced              |`fencing`          | `error`  | `hostname`
>> +| Storage replication job failed   |`replication`      | `error`  | `hostname`, `job-id`
>> +| Backup succeeded                 |`vzdump`           | `info`   | `hostname`, `job-id` (only for backup jobs)
>> +| Backup failed                    |`vzdump`           | `error`  | `hostname`, `job-id` (only for backup jobs)
> 
> I think the "(only for backup jobs)" should not be there, or?
> 

No, that should be there. The `job-id` field is set if the backup is started
as a scheduled job and if that job is started by hand, but *not* if
e.g. a single VM is backed up manually. Would not make sense for the latter,
since that one has no associated job.

-- 
- Lukas