getsentry / getsentry/sentry-python

No Quota Separation in Transport Layer

Ouverte
#3,575 3 commentaires 0 réactions 1 personne assignée Réclamée par @sl0thentr0py Voir sur GitHub
Component: Backpressure Monitor Feature Python Quality Improvement
Langage dominant
Python
Étoiles
2.2k
Forks
669
Merge moyen
1 j 1 h
PR mergées (30 j)
213

Description

### How do you use Sentry?

Sentry Saas (sentry.io)

### Version

2.9.0

### Steps to Reproduce

1. Exhaust performance units quota

![Image](https://github.com/user-attachments/assets/1f5bc8ba-0f30-4636-80cf-87ff93bd256b)

2. Attempt to send performance metrics, and you will receive rate limiting issues:
```
[sentry] WARNING: Rate-limited via x-sentry-rate-limits
[sentry] DEBUG: [Monitor] health check negative, downsampling with a factor of 1
```

3. Trigger an error event, it will incorrectly be impacted by the rate limits
```
[sentry] INFO: event processor (.processor at 0xffffa2193560>) dropped event
```

### Expected Result

The Sentry SDK needs a more specific approach to handling rate limits for different event types, such that performance metrics are rate limited when the quota is expended, but error events are still sent to Sentry.

### Actual Result

```
KeyError: 'TriggerTestEvent'
[sentry] INFO: event processor (.processor at 0xffffa2193560>) dropped event
[sentry] DEBUG: [Monitor] health check negative, downsampling with a factor of 2
[sentry] DEBUG: [Monitor] health check negative, downsampling with a factor of 3
[sentry] DEBUG: [Monitor] health check negative, downsampling with a factor of 4
[sentry] DEBUG: [Monitor] health check negative, downsampling with a factor of 5
[sentry] DEBUG: [Monitor] health check negative, downsampling with a factor of 6
```

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.