mirror of
https://github.com/NousResearch/hermes-agent.git
synced 2026-07-31 19:16:29 +00:00
fix(monitoring): surface cron snapshot failures at WARNING (content-free)
The cron health snapshot failure path logged only at DEBUG, so a cron telemetry regression would silently drop all hermes.cron.* metrics while gateway health metrics kept flowing. Promote to WARNING with the exception *type* name only (no message text, preserving the no-raw-error contract); keep the traceback on DEBUG. (cherry picked from commit 1c9a3e737b7c57fa9886bc927f1623753a9a811b)
This commit is contained in:
parent
58a7491ecd
commit
6de6e36dbe
1 changed files with 10 additions and 2 deletions
|
|
@ -305,8 +305,16 @@ def _read_runtime_snapshot(config: Dict[str, Any]):
|
|||
gateway_snapshot = _read_gateway_snapshot(config)
|
||||
try:
|
||||
cron_snapshot = _read_cron_snapshot()
|
||||
except Exception:
|
||||
logger.debug("cron health snapshot unavailable", exc_info=True)
|
||||
except Exception as exc:
|
||||
# Content-free visibility: cron telemetry silently dropping out is a
|
||||
# release-relevant regression, so surface it at WARNING with only the
|
||||
# exception *type* name (never the message, which could carry paths or
|
||||
# other environment detail). exc_info stays on the DEBUG record.
|
||||
logger.warning(
|
||||
"cron health snapshot unavailable; cron telemetry not exported (error_type=%s)",
|
||||
type(exc).__name__,
|
||||
)
|
||||
logger.debug("cron health snapshot traceback", exc_info=True)
|
||||
return gateway_snapshot
|
||||
gateway_snapshot.metrics.extend(cron_snapshot.metrics)
|
||||
return gateway_snapshot
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue