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

[Fiona Ebner]

Am 22.04.24 um 11:15 schrieb 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.
> 

Oh, sorry. I had misinterpreted this and didn't think about manual
invocation vs. job.