Skip to content

Conversation

@wojcik-dorota
Copy link
Collaborator

@wojcik-dorota wojcik-dorota commented Dec 12, 2025

https://aiven.atlassian.net/browse/MA-3655

DOC PREVIEW

  • The first paragraph of the page is on one line.
  • The other lines have a line break at 90 characters.
  • I checked the output.
  • I applied the style guide.
  • My links start with /docs/.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Dec 12, 2025

Deploying aiven-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 63c5fb2
Status: ✅  Deploy successful!
Preview URL: https://66ed0bec.aiven-docs.pages.dev
Branch Preview URL: https://ma-3655-docs-updates-to-the.aiven-docs.pages.dev

View logs

@wojcik-dorota wojcik-dorota force-pushed the MA-3655-docs-updates-to-the-open-search-log-integration-doc branch from fc292f2 to 55b3def Compare December 12, 2025 13:06
@wojcik-dorota wojcik-dorota force-pushed the MA-3655-docs-updates-to-the-open-search-log-integration-doc branch from 55b3def to b0a52e2 Compare December 12, 2025 13:33
@wojcik-dorota wojcik-dorota marked this pull request as ready for review December 12, 2025 14:26
@wojcik-dorota wojcik-dorota requested a review from a team as a code owner December 12, 2025 14:26
@wojcik-dorota wojcik-dorota self-assigned this Dec 15, 2025
Copy link

@ArthurAnsquer ArthurAnsquer left a comment

Choose a reason for hiding this comment

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

From my non-technical point of view, this is exactly what we were lacking, so this is great! The console part looks also good to me, but can't comment on the API/TF/K8 part - maybe we get someone else more technical for an opinion? :)
Great job!

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the OpenSearch log integration documentation by expanding it from basic console-only instructions to comprehensive multi-platform guidance. The update adds tabbed sections covering the Aiven Console, API, CLI, Terraform, and Kubernetes methods for enabling, configuring, and disabling log integrations. The PR also adds cross-references from related documentation pages to improve discoverability.

Key changes:

  • Adds prerequisites section with platform-specific requirements across five tabs (Console, API, CLI, Terraform, Kubernetes)
  • Expands enable, configure, and disable sections with detailed examples for each platform
  • Adds internal cross-references from related log integration documentation

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
docs/products/opensearch/howto/opensearch-log-integration.md Main update: expands from console-only to multi-platform documentation with code examples for API, CLI, Terraform, and Kubernetes
docs/products/kafka/howto/integrate-service-logs-into-kafka-topic.md Adds cross-reference link to the updated OpenSearch log integration guide
docs/products/grafana.md Converts OpenSearch mention to hyperlink referencing the log integration guide
docs/products/flink.md Converts existing log integration text to hyperlink
docs/platform/howto/list-monitoring.md Converts inline text about log integration to hyperlink
docs/platform/concepts/service-integration.md Adds hyperlink to log integration documentation
docs/integrations/send-logs-to-elasticsearch.md Adds cross-reference link to OpenSearch log integration guide

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 14 comments.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 6 comments.

@wojcik-dorota wojcik-dorota merged commit 5d5e329 into main Dec 18, 2025
5 checks passed
@wojcik-dorota wojcik-dorota deleted the MA-3655-docs-updates-to-the-open-search-log-integration-doc branch December 18, 2025 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants