Skip to content

[AIG] Logpush initial documentation #16798

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 11 commits into from
Sep 26, 2024
Merged

[AIG] Logpush initial documentation #16798

merged 11 commits into from
Sep 26, 2024

Conversation

@daisyfaithauma
Copy link
Contributor

@daisyfaithauma daisyfaithauma commented Sep 12, 2024 •
edited
Loading

  • Logpush initial documentation

@daisyfaithauma daisyfaithauma changed the title Logpush initial documentation [AIG] Logpush initial documentation Sep 12, 2024
@github-actions github-actions bot added product:ai-gateway AI Gateway: https://developers.cloudflare.com/ai-gateway/ size/m labels Sep 12, 2024
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Sep 12, 2024 •
edited
Loading

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1f2ea21
Status: ✅  Deploy successful!
Preview URL: https://a54af775.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://aig-logpush.cloudflare-docs-7ou.pages.dev

View logs

@github-actions
Copy link
Contributor

Files with changes (up to 15)

Original Link Updated Link
https://developers.cloudflare.com/ai-gateway/observability/logpush/ https://aig-logpush.cloudflare-docs-7ou.pages.dev/ai-gateway/observability/logpush/

@kathayl
Copy link
Contributor

kathayl commented Sep 12, 2024

oh i did not know we could have a "draft pull request" status. Great!
Thinking this might be better as a subpage under the (future) logging section? So like:

Observability
-Analytics
-Logging
---Logpush
-Costs

What do you think?

kathayl
kathayl reviewed Sep 24, 2024
View reviewed changes
src/content/docs/ai-gateway/observability/logpush.mdx Outdated

AI Gateway allows you to securely push logs to an external storage location, where you can decrypt and process them. This guide explains how to set up Logpush for the AI Gateway, generate an RSA key pair for encryption, and decrypt the logs once they are received.
AI Gateway allows you to securely export logs to an external storage location, where you can decrypt and process them. This guide explains how to set up Logpush for AI Gateway, generate an RSA key pair for encryption, and decrypt the logs once they are received.
Logpush has a limit of 4 jobs.
Copy link
Contributor

Choose a reason for hiding this comment

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

  1. should this be a note?
  2. Can we add "If you have reached the limit of logs stored for your plan (insert link to limits page), logs will not be exported through Logpush.

Copy link
Contributor Author

@daisyfaithauma daisyfaithauma Sep 25, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

  1. should this be a note?
  2. Can we add "If you have reached the limit of logs stored for your plan (insert link to limits page), logs will not be exported through Logpush.
  1. I do not think so. It explains what the guide does.
  2. Done

@daisyfaithauma daisyfaithauma marked this pull request as ready for review September 26, 2024 10:33
Oxyjun
Oxyjun reviewed Sep 26, 2024
View reviewed changes
src/content/docs/ai-gateway/observability/logpush.mdx
import { Render } from "~/components";

AI Gateway allows you to securely export logs to an external storage location, where you can decrypt and process them.
You can toggle Logpush on and off in the [Cloudflare dashboard](https://dash.cloudflare.com) settings.
Copy link
Contributor

Choose a reason for hiding this comment

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

Not a suggestion but - does this linebreak even get rendered in the docs? Or does MDX ignore it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

mdx ignores it. This is mainly for me to know what I am writing lol

Oxyjun
Oxyjun reviewed Sep 26, 2024
View reviewed changes
Oxyjun
Oxyjun reviewed Sep 26, 2024
View reviewed changes
Oxyjun
Oxyjun reviewed Sep 26, 2024
View reviewed changes
Oxyjun
Oxyjun reviewed Sep 26, 2024
View reviewed changes
Oxyjun
Oxyjun reviewed Sep 26, 2024
View reviewed changes
@daisyfaithauma daisyfaithauma merged commit fbf061f into production Sep 26, 2024
14 checks passed
@daisyfaithauma daisyfaithauma deleted the aig-logpush branch September 26, 2024 11:12
patriciasantaana pushed a commit that referenced this pull request Sep 26, 2024
@daisyfaithauma @kathayl
@Oxyjun @patriciasantaana
[AIG] Logpush initial documentation (#16798)
a226aaf 
* Logpush initial documentation

* Update logpush.mdx

* Update logpush.mdx

* Added limits link

* Updated details

* Add script to run file

* Update logpush.mdx

* Update src/content/docs/ai-gateway/observability/logpush.mdx

Replace

Co-authored-by: Jun Lee <[email protected]>

* Update src/content/docs/ai-gateway/observability/logpush.mdx

Duplicate

Co-authored-by: Jun Lee <[email protected]>

* Update src/content/docs/ai-gateway/observability/logpush.mdx

Co-authored-by: Jun Lee <[email protected]>

* Update src/content/docs/ai-gateway/observability/logpush.mdx

Co-authored-by: Jun Lee <[email protected]>

---------

Co-authored-by: Kathy <[email protected]>
Co-authored-by: Jun Lee <[email protected]>
elithrar pushed a commit that referenced this pull request Oct 15, 2024
@daisyfaithauma @kathayl
@Oxyjun @elithrar
[AIG] Logpush initial documentation (#16798)
eae008d 
* Logpush initial documentation

* Update logpush.mdx

* Update logpush.mdx

* Added limits link

* Updated details

* Add script to run file

* Update logpush.mdx

* Update src/content/docs/ai-gateway/observability/logpush.mdx

Replace

Co-authored-by: Jun Lee <[email protected]>

* Update src/content/docs/ai-gateway/observability/logpush.mdx

Duplicate

Co-authored-by: Jun Lee <[email protected]>

* Update src/content/docs/ai-gateway/observability/logpush.mdx

Co-authored-by: Jun Lee <[email protected]>

* Update src/content/docs/ai-gateway/observability/logpush.mdx

Co-authored-by: Jun Lee <[email protected]>

---------

Co-authored-by: Kathy <[email protected]>
Co-authored-by: Jun Lee <[email protected]>
maheshwarip pushed a commit that referenced this pull request Dec 2, 2024
@daisyfaithauma @kathayl
@Oxyjun @maheshwarip
[AIG] Logpush initial documentation (#16798)
507595b 
* Logpush initial documentation

* Update logpush.mdx

* Update logpush.mdx

* Added limits link

* Updated details

* Add script to run file

* Update logpush.mdx

* Update src/content/docs/ai-gateway/observability/logpush.mdx

Replace

Co-authored-by: Jun Lee <[email protected]>

* Update src/content/docs/ai-gateway/observability/logpush.mdx

Duplicate

Co-authored-by: Jun Lee <[email protected]>

* Update src/content/docs/ai-gateway/observability/logpush.mdx

Co-authored-by: Jun Lee <[email protected]>

* Update src/content/docs/ai-gateway/observability/logpush.mdx

Co-authored-by: Jun Lee <[email protected]>

---------

Co-authored-by: Kathy <[email protected]>
Co-authored-by: Jun Lee <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kathayl kathayl kathayl approved these changes

@Oxyjun Oxyjun Oxyjun approved these changes

@G4brym G4brym Awaiting requested review from G4brym

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

@pdwittig pdwittig Awaiting requested review from pdwittig

Assignees

@pdwittig pdwittig

@G4brym G4brym

@mchenco mchenco

@kathayl kathayl

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@daisyfaithauma @kathayl @Oxyjun @pdwittig @G4brym @mchenco