grafana / grafana/pyroscope

Add profile push limits

Open
#2,214 2 comments 1 reaction 1 assignee Claimed by @kolesnikovae View on GitHub
enhancement storage
Dominant language
Go
Stars
11.7k
Forks
802
Avg merge
1d 19h
Merged PRs (30d)
80

Description

The following tenant-level limits should be added in order to protect the write path from misuse/abuse:

1. `max_local_symbols_stacktraces_per_tenant` (~16M default)
2. `max_local_symbols_locations_per_tenant` (~4M default) (plus `functions` and `mappings`, optionally)
3. `max_local_symbols_strings_size_per_tenant` (~64MB default)

Optionally, each of them may have a **global** counterpart.

See https://github.com/grafana/pyroscope/pull/2157 for the context.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.