Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(crons): Add queue_incident_occurrence #80842

Merged

Conversation

evanpurkhiser
Copy link
Member

@evanpurkhiser evanpurkhiser commented Nov 15, 2024

Produces an IncidentOccurrence message into the monitors-incident-occurrences topic for later issue occurrence dispatching.

Part of GH-79328

Produces an IncidentOccurrence message into the
monitors-incident-occurrences topic for later issue occurrence
dispatching.
Copy link

codecov bot commented Nov 15, 2024

Codecov Report

Attention: Patch coverage is 69.56522% with 7 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/monitors/logic/incident_occurrence.py 69.56% 6 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #80842      +/-   ##
==========================================
- Coverage   78.42%   78.41%   -0.01%     
==========================================
  Files        7209     7209              
  Lines      319618   319641      +23     
  Branches    44006    44007       +1     
==========================================
+ Hits       250646   250653       +7     
- Misses      62586    62598      +12     
- Partials     6386     6390       +4     

"clock_tick_ts": int(clock_tick.timestamp()),
}

# The incident occurrence should be
Copy link
Member

Choose a reason for hiding this comment

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

please finish comment

Copy link
Member

@JoshFerge JoshFerge left a comment

Choose a reason for hiding this comment

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

looks good. follow ups for spelling mistake + comments

@evanpurkhiser evanpurkhiser merged commit fc8e89c into master Nov 15, 2024
51 of 52 checks passed
@evanpurkhiser evanpurkhiser deleted the evanpurkhiser/feat-crons-add-queue-incident-occurrence branch November 15, 2024 20:44
evanpurkhiser added a commit that referenced this pull request Nov 15, 2024
Passes the clock_tick timestamp through mark_failed, which
dispatch_incident_occurrence will used to decide how to dispatch the
incident_occurrence (through queuing, or directly sending)

Requires #80842
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