Skip to content

Commit e789985

Browse files
committed
move hard limits to open questions
Signed-off-by: Bogdan Stancu <[email protected]>
1 parent bd2ca17 commit e789985

File tree

1 file changed

+6
-21
lines changed

1 file changed

+6
-21
lines changed

docs/proposals/limits-api.md renamed to docs/proposals/user-overrides-api.md

Lines changed: 6 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -64,29 +64,14 @@ Removes tenant-specific overrides, reverting to default overrides.
6464
2. Security:
6565
- Rate limiting will be implemented to prevent abuse
6666
- Changes will be validated before being applied
67-
- A hard limit configuration will be implemented
68-
Hard limits will not be changable through the API
69-
Example:
70-
```yaml
71-
# file: runtime.yaml
72-
# In this example, we're overriding ingestion limits for a single tenant.
73-
overrides:
74-
"user1":
75-
ingestion_burst_size: 350000
76-
ingestion_rate: 350000
77-
max_global_series_per_metric: 300000
78-
max_global_series_per_user: 300000
79-
max_series_per_metric: 0
80-
max_series_per_user: 0
81-
max_samples_per_query: 100000
82-
max_series_per_query: 100000
83-
configurable-overrides: # still not sure about the naming for this section
84-
"user1":
85-
ingestion_rate: 700000
86-
max_global_series_per_user: 700000
87-
```
67+
8868

8969
3. Error Handling:
9070
- Invalid limit values will return 400 Bad Request
9171
- Storage backend errors will return 500 Internal Server Error
9272

73+
### Open Questions:
74+
- How do we implement a hard-limit configuration to avoid users
75+
setting unreasonable limits?
76+
- What set of overrides can be configurable through this API?
77+
Limits like `shard_size` should only be modified by the admin.

0 commit comments

Comments
 (0)