Skip to content

Update limitations for token usage #2382

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Aug 5, 2025
Merged

Update limitations for token usage #2382

merged 6 commits into from
Aug 5, 2025

Conversation

shubhaat
Copy link
Contributor

@shubhaat shubhaat commented Aug 1, 2025

Update the token usage limitation to include fair use policy

Update the token usage limitation to include fair use policy
@shubhaat shubhaat self-assigned this Aug 1, 2025
@shubhaat shubhaat marked this pull request as ready for review August 1, 2025 22:41
@shubhaat shubhaat requested a review from a team as a code owner August 1, 2025 22:41
Copy link

github-actions bot commented Aug 1, 2025

🔍 Preview links for changed docs

@shubhaat shubhaat requested a review from ppf2 August 1, 2025 22:43
@shubhaat shubhaat assigned demjened and unassigned demjened Aug 1, 2025
@shubhaat shubhaat requested a review from demjened August 1, 2025 22:44
Copy link
Contributor

@benironside benironside left a comment

Choose a reason for hiding this comment

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

Left a couple small suggestions. Thanks for submitting this!

@shubhaat
Copy link
Contributor Author

shubhaat commented Aug 1, 2025

Thanks for the quick review @benironside , I accepted and applied them to both places. For now, I would suggest we merge this when it works for us. This should apply to serverless and hosted, if we can merge it with 9.1+ that would be perfect for hosted.

@shubhaat shubhaat requested review from benironside and removed request for ppf2 and demjened August 4, 2025 14:51
@shubhaat
Copy link
Contributor Author

shubhaat commented Aug 5, 2025

@benironside or someone on @elastic/admin-docs , if you can help me understand if this can be backported to 8.19/9.1 and 8.18/9.0?

Copy link
Contributor

@szabosteve szabosteve left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@yetanothertw yetanothertw left a comment

Choose a reason for hiding this comment

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

Looks good, thank you! 🐱

@szabosteve szabosteve merged commit 52dd675 into main Aug 5, 2025
8 checks passed
@szabosteve szabosteve deleted the llm-usage-limits-1 branch August 5, 2025 10:12
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.

5 participants