From 114c4ced3d76a047c89e098057dc7d8d5a448298 Mon Sep 17 00:00:00 2001 From: Kriegsameise Date: Fri, 13 Mar 2026 14:20:30 +0100 Subject: [PATCH] Fix crash from possible negative time Possible negative value for backup age while backup is running, dont yield if it's negative --- .../agent_based/proxmox_ve_backup_status.py | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/packages/cmk-plugins/cmk/plugins/proxmox_ve/agent_based/proxmox_ve_backup_status.py b/packages/cmk-plugins/cmk/plugins/proxmox_ve/agent_based/proxmox_ve_backup_status.py index dcf4a74cb6b..e8b5388cb9a 100644 --- a/packages/cmk-plugins/cmk/plugins/proxmox_ve/agent_based/proxmox_ve_backup_status.py +++ b/packages/cmk-plugins/cmk/plugins/proxmox_ve/agent_based/proxmox_ve_backup_status.py @@ -115,14 +115,16 @@ def check_proxmox_ve_vm_backup_status( # explicitly converted them to utc started_time = last_backup.get("started_time") if started_time: - yield from check_levels( - value=(now - started_time.astimezone(UTC)).total_seconds(), - levels_upper=params["age_levels_upper"], - metric_name="age", - render_func=render.timespan, - label="Age", - boundaries=(0, None), - ) + age = (now - started_time.astimezone(UTC)).total_seconds() + if age >= 0: + yield from check_levels( + value=age, + levels_upper=params["age_levels_upper"], + metric_name="age", + render_func=render.timespan, + label="Age", + boundaries=(0, None), + ) yield Result( state=State.OK, summary=f"Server local start time: {started_time}",