Skip to content

chore(task-processor): Send fingerprint to change Sentry grouping - #280

Closed
matthewelwell wants to merge 1 commit into
mainfrom
chore/alert-in-sentry-for-each-failed-task
Closed

matthewelwell wants to merge 1 commit into
mainfrom
chore/alert-in-sentry-for-each-failed-task

Conversation

@matthewelwell

Copy link
Copy Markdown
Contributor

Adds a custom fingerprint to the error message sent when we automatically disable a recurring task. This ensures that Sentry alerts for each individual task instead of grouping alerts across all recurring tasks.

@matthewelwell
matthewelwell requested a review from a team as a code owner October 19, 2026 13:52
@matthewelwell
matthewelwell requested review from khvn26 and removed request for a team October 19, 2026 13:52
@coderabbitai

coderabbitai Bot commented Oct 19, 2026

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 6af5e090-ce03-470e-a662-bd014adf7425
📥 Commits

Reviewing files that changed from the base of the PR and between 5c14153 and 5b01a9f.

📒 Files selected for processing: 1
  • src/task_processor/models.py
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.66%. Comparing base (5c14153) to head (5b01a9f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #280   +/-   ##
=======================================
  Coverage   97.66%   97.66%           
=======================================
  Files         110      110           
  Lines        4963     4963           
=======================================
  Hits         4847     4847           
  Misses        116      116           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@matthewelwell

Copy link
Copy Markdown
Contributor Author

Solved via a new alert rule in sentry -> slack confif

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants