getsentry / getsentry/sentry-python

No Quota Separation in Transport Layer

Aberta
#3,575 3 comentários 0 reações 1 responsável Reivindicada por @sl0thentr0py Ver no GitHub
Component: Backpressure Monitor Feature Python Quality Improvement
Linguagem predominante
Python
Estrelas
2.2k
Forks
669
Merge médio
1d 40min
PRs com merge (30d)
212

Descrição

### 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
```

Guia de contribuição

Abrir o guia de contribuição

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.