Skip to content

feat(seer): Add logging and metrics for Seer entrypoints#107693

Open
leeandher wants to merge 2 commits intomasterfrom
leanderrodrigues/iswf-1158-implement-logs-and-metrics-with-dashboards
Open

feat(seer): Add logging and metrics for Seer entrypoints#107693
leeandher wants to merge 2 commits intomasterfrom
leanderrodrigues/iswf-1158-implement-logs-and-metrics-with-dashboards

Conversation

@leeandher
Copy link
Member

Add logging and metrics instrumentation to the Seer operator, cache, and Slack entrypoint modules.

Refs ISWF-1158

Add comprehensive logging and metrics instrumentation to the Seer
operator, cache, and Slack entrypoint modules to improve observability
of autofix workflow triggers and updates.

- Add metrics for autofix triggers, cache migrations, and thread updates
- Standardize log message names with seer.* prefix convention
- Use logger.exception() within exception handlers for tracebacks
- Enrich logging context with stopping points, run IDs, and thread counts

Refs ISWF-1158
Co-Authored-By: Claude <noreply@anthropic.com>
@linear
Copy link

linear bot commented Feb 5, 2026

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Feb 5, 2026
The group_not_found case is inside an exception handler, so the test
should expect logger.exception instead of logger.error.
@leeandher leeandher marked this pull request as ready for review February 5, 2026 19:38
@leeandher leeandher requested review from a team as code owners February 5, 2026 19:38
Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

_logger.info("seer.slack.trigger_autofix.complete", extra=logging_ctx)
metrics.incr(
"seer.slack.trigger_autofix",
tags={"stopping_point": str(stopping_point), "is_continuation": str(is_continuation)},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you need to stringify a bool (is_continuation)? I wasn't sure so I looked up a random example and it seems ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants