Skip to content

Conversation

@kcons
Copy link
Member

@kcons kcons commented Jun 6, 2025

Continuing with partial data is less correct than retrying the task.

@kcons kcons requested a review from a team as a code owner June 6, 2025 00:17
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 6, 2025
@codecov
Copy link

codecov bot commented Jun 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #93002   +/-   ##
=======================================
  Coverage   87.90%   87.90%           
=======================================
  Files       10289    10289           
  Lines      590634   590703   +69     
  Branches    22965    22965           
=======================================
+ Hits       519175   519241   +66     
- Misses      71012    71015    +3     
  Partials      447      447           

@kcons
Copy link
Member Author

kcons commented Jun 6, 2025

oop, mypy error, but a trivial one.

@kcons kcons merged commit 8ef5db1 into master Jun 6, 2025
61 checks passed
@kcons kcons deleted the kcons/failloud branch June 6, 2025 18:27
@sentry
Copy link

sentry bot commented Jun 8, 2025

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ SnubaError: HTTPConnectionPool(host='snuba-api', port=80): Read timed out. (read timeout=30) sentry.workflow_engine.processors.delayed_workflow View Issue

Did you find this useful? React with a 👍 or 👎

andrewshie-sentry pushed a commit that referenced this pull request Jun 19, 2025
Continuing with partial data is less correct than retrying the task.
@github-actions github-actions bot locked and limited conversation to collaborators Jun 23, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants