Skip to content

docs: sync new burst curve + gRPC endpoints#2662

Open
tricky42 wants to merge 1 commit into
masterfrom
abu/rl-v2-increased-burst
Open

docs: sync new burst curve + gRPC endpoints#2662
tricky42 wants to merge 1 commit into
masterfrom
abu/rl-v2-increased-burst

Conversation

@tricky42

@tricky42 tricky42 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

What

  • Sync src/lib/rate-limits/data/bucket-to-threshold.csv with the reworked rpm-v2 burst curve — an anchored ratio declining 6.0×@100 → 3.0×@15000 → 1.4×@80000 (10 rps floor ≤100 rpm; was flat ~2.4×). rps is snapped up (ceil) per rung so no rung dips below the target line, so a few low rungs sit just above 6× (e.g. 175rpm→18rps=6.17×); burst is monotonic non-decreasing.
  • Sync bucket-to-endpoints.csv from the canonical generator (tools/ratelimit-csvs): includes the Keto gRPC endpoints (ory.keto.relation_tuples.*Service/*) and the Talos REST admin endpoints (/v2alpha1/admin/...), plus /csp-report.
  • Regenerate src/static/rate-limits.json.

Why

Keeps the docs rate-limit reference in sync with the deployed config; reflects the burst-curve change and enterprise-enforce rollout in ory-corp/cloud#12683. Supersedes #2627 (whose Talos/Keto endpoints are now produced canonically).

… new burst curve

Pull the canonical bucket-to-threshold.csv (new rpm-v2 burst rps: 6x@100 ->
3x@15k -> 1.4x@80k) and bucket-to-endpoints.csv from cloud tools/ratelimit-csvs,
and rebuild src/static/rate-limits.json.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 5, 2026 13:22
@tricky42 tricky42 changed the title docs(rate-limits): sync new burst curve + gRPC endpoints docs: sync new burst curve + gRPC endpoints Jul 5, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the docs rate-limit reference data to match the latest deployed configuration by refreshing the burst (rps) curve derived from rpm and regenerating the endpoint/bucket mapping + exported JSON artifact.

Changes:

  • Updated bucket-to-threshold.csv rps values to reflect the new rpm-v2 burst curve.
  • Updated bucket-to-endpoints.csv with a new /csp-report endpoint mapping.
  • Regenerated src/static/rate-limits.json from the updated CSV inputs.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
src/static/rate-limits.json Regenerated static JSON export reflecting updated endpoints and thresholds.
src/lib/rate-limits/data/bucket-to-threshold.csv Updated rps column across all tiers/envs per the new burst curve.
src/lib/rate-limits/data/bucket-to-endpoints.csv Added /csp-report endpoint mapping for Kratos public high bucket.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/lib/rate-limits/data/bucket-to-endpoints.csv
Comment thread src/lib/rate-limits/data/bucket-to-threshold.csv
@coderabbitai

coderabbitai Bot commented Jul 5, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

This change updates the rate-limiting configuration file, revising rps/rpm threshold values across all bucket, tier, and environment combinations, while leaving the endpoints array unchanged.

Changes

Rate limit configuration update

Layer / File(s) Summary
Threshold value updates
src/static/rate-limits.json
Updated rps/rpm threshold values for all bucket/tier/environment combinations; endpoints entries unchanged.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

  • ory/docs#2638: Also modifies src/static/rate-limits.json thresholds (rps/rpm) for rate-limit buckets/tiers/environments.

Suggested reviewers: vinckr, aeneasr, zepatrik

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly matches the main change by mentioning the burst curve and gRPC endpoint sync.
Description check ✅ Passed The description covers what changed and why, with issue references, but it omits explicit checklist completion and some template sections.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch abu/rl-v2-increased-burst

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants