Skip to content

Update caching.mdx #18004

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 9 commits into from
Nov 8, 2024
Merged

Update caching.mdx #18004

merged 9 commits into from
Nov 8, 2024

Conversation

@kathayl
Copy link
Contributor

@kathayl kathayl commented Nov 5, 2024

caching updates

Summary

Screenshots (optional)

Documentation checklist

  • The documentation style guide has been adhered to.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

@kathayl kathayl requested review from a team, G4brym, daisyfaithauma and mchenco as code owners November 5, 2024 23:33
@github-actions github-actions bot added product:ai-gateway AI Gateway: https://developers.cloudflare.com/ai-gateway/ size/s labels Nov 5, 2024
@kathayl
Copy link
Contributor Author

kathayl commented Nov 5, 2024 •
edited
Loading

I've updated the page with

  1. the new updated headers
  2. the minimum time to live of 60 seconds and max of 1 month
  3. changed the example from milliseconds to seconds

Daisy, can you

  1. figure out how the best way to say that the old headers still work but we have new names. While the old headers will still work, we recommend updating to the new headers to ensure compatibility with future releases:
  • cf-cache-ttl has been replaced by cf-aig-cache-ttl
  • cf-skip-cache has been replaced by cf-aig-skip-cache
  1. need to add this note in about caching behavior
    Note: Cache in AI Gateway is volatile. If two identical requests are sent simultaneously, the first request may not cache in time for the second to use it, causing the second request to retrieve data from the original source.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 5, 2024 •
edited
Loading

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2dd89df
Status: ✅  Deploy successful!
Preview URL: https://c0e26026.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://kathayl-patch-1.cloudflare-docs-7ou.pages.dev

View logs

@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2024 •
edited
Loading

Files with changes (up to 15)

Original Link Updated Link
https://developers.cloudflare.com/ai-gateway/configuration/caching/ https://kathayl-patch-1.cloudflare-docs-7ou.pages.dev/ai-gateway/configuration/caching/
https://developers.cloudflare.com/ai-gateway/reference/limits/ https://kathayl-patch-1.cloudflare-docs-7ou.pages.dev/ai-gateway/reference/limits/

hyperlint-ai[bot]
hyperlint-ai bot reviewed Nov 8, 2024
View reviewed changes
Copy link
Contributor

@hyperlint-ai hyperlint-ai bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 1 total issue(s) found.

@daisyfaithauma @hyperlint-ai
Update src/content/docs/ai-gateway/reference/limits.mdx
f44de56 
Co-authored-by: hyperlint-ai[bot] <154288675+hyperlint-ai[bot]@users.noreply.github.com>
hyperlint-ai[bot]
hyperlint-ai bot reviewed Nov 8, 2024
View reviewed changes
Copy link
Contributor

@hyperlint-ai hyperlint-ai bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 1 total issue(s) found.

pedrosousa
pedrosousa reviewed Nov 8, 2024
View reviewed changes
pedrosousa
pedrosousa reviewed Nov 8, 2024
View reviewed changes
@kathayl kathayl merged commit 28a0806 into production Nov 8, 2024
13 checks passed
@kathayl kathayl deleted the kathayl-patch-1 branch November 8, 2024 18:22
harshil1712 pushed a commit that referenced this pull request Dec 3, 2024
@kathayl @daisyfaithauma
@hyperlint-ai @harshil1712
Update caching.mdx (#18004)
4fc29f1 
* Update limits.mdx

* Edited footnotes

* Update caching.mdx

caching updates

* Update caching.mdx

* Update src/content/docs/ai-gateway/reference/limits.mdx

Co-authored-by: hyperlint-ai[bot] <154288675+hyperlint-ai[bot]@users.noreply.github.com>

* merge conflict

* merge conflict

* merge conflict

---------

Co-authored-by: daisyfaithauma <[email protected]>
Co-authored-by: hyperlint-ai[bot] <154288675+hyperlint-ai[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pedrosousa pedrosousa pedrosousa left review comments

@G4brym G4brym Awaiting requested review from G4brym

@mchenco mchenco Awaiting requested review from mchenco mchenco is a code owner

+2 more reviewers

@hyperlint-ai hyperlint-ai[bot] hyperlint-ai[bot] left review comments

@daisyfaithauma daisyfaithauma daisyfaithauma approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@G4brym G4brym

@mchenco mchenco

@daisyfaithauma daisyfaithauma

Labels

product:ai-gateway AI Gateway: https://developers.cloudflare.com/ai-gateway/ size/s

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@kathayl @pedrosousa @daisyfaithauma @G4brym @mchenco