Skip to content

feat: support weekly, monthly, and yearly quota windows - #2662

Closed
fernandoescolar wants to merge 1 commit into
theagentrouter:mainfrom
fernandoescolar:main
Closed

fernandoescolar wants to merge 1 commit into
theagentrouter:mainfrom
fernandoescolar:main

Conversation

@fernandoescolar

@fernandoescolar fernandoescolar commented Sep 9, 2026

Copy link
Copy Markdown

Description

This PR extends QuotaPolicy duration support to include weekly, monthly, and yearly quota windows:

  • 1w for one week
  • 1mo for one month
  • 1y for one year

The Envoy Rate Limit Service API now exposes the corresponding WEEK, MONTH and YEAR rate limit units. This allows Agent Router users to configure longer-lived token quotas for weekly budgets, monthly spend limits, and yearly
quotas.

The CRD validation, generated Helm manifest, documentation, translator, and tests have been updated accordingly.

Related Issues/PRs

Related issue: [1]
Related PR: [2]

Special notes for reviewers

The Envoy Rate Limit Service API supports fixed units, not arbitrary duration multipliers. Its configuration uses requests_per_unit and a single rate limit unit, so formats such as 5d, 30d, or 365d cannot be represented directly.

For that reason, this PR supports the native Envoy units 1w, 1mo, and 1y instead of adding multiplier-based formats.

These units represent Envoy rate-limit windows and are not calendar-aware billing periods.

1: #2469
2: envoyproxy/go-control-plane#1223

@fernandoescolar
fernandoescolar requested a review from a team as a code owner September 9, 2026 19:15
@netlify

netlify Bot commented Sep 9, 2026

Copy link
Copy Markdown

Deploy Preview for theagentrouter canceled.

Name Link
🔨 Latest commit 0dfd66f
🔍 Latest deploy log https://app.netlify.com/projects/theagentrouter/deploys/6aa1b202cc45e900081c3619

Signed-off-by: Fernando Escolar <fernando.escolar@digits.schwarz>
@fernandoescolar

Copy link
Copy Markdown
Author

Closing in favor of #2695, recreated from a dedicated branch (feat/quota-windows) instead of main.

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.

1 participant