Skip to content

Conversation

florence-crl
Copy link
Contributor

@florence-crl florence-crl commented Sep 10, 2025

Fixes DOC-14144 DOC-14437 DOC-14537 DOC-8496 DOC-14810

  • In configure-logs.md, (a) added sections for Output to OTLP-compatible network collectors and Set OTLP defaults (b) updated default log config.
  • In logging-use-cases.md, modified example for Network logging with log buffering to use OTLP instead of Fluentd.

Rendered preview

…twork collectors and Set OTLP.

In logging-use-cases.md, modified example for Network logging with log buffering to use OTLP instead of Fluentd.
Copy link

Copy link

netlify bot commented Sep 10, 2025

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit fbcf137
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/68c4873b2703050008b9dc48

Copy link

netlify bot commented Sep 10, 2025

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit fbcf137
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/68c4873bfa504b00085be040

@florence-crl
Copy link
Contributor Author

Hi @arjunmahishi, I assigned you to do this docs review since you reviewed Mudit Somani’s code prs. tia

Copy link

netlify bot commented Sep 10, 2025

Netlify Preview

Name Link
🔨 Latest commit fbcf137
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/68c4873b87d8aa00087949ed
😎 Deploy Preview https://deploy-preview-20320--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

@arjunmahishi arjunmahishi left a comment

Choose a reason for hiding this comment

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

Thank you @florence-crl!
I've left one suggestion. Rest looks good to me.

Copy link

@arjunmahishi arjunmahishi left a comment

Choose a reason for hiding this comment

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

LGTM

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.

2 participants