Skip to content

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 demjened and ppf2 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