-
Notifications
You must be signed in to change notification settings - Fork 25
update: os log integration #1220
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.
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
update: os log integration #1220
Conversation
Deploying aiven-docs with
|
| 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 |
fc292f2 to
55b3def
Compare
55b3def to
b0a52e2
Compare
ArthurAnsquer
left a comment
There was a problem hiding this 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!
There was a problem hiding this 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 |
There was a problem hiding this 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.
There was a problem hiding this 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.
https://aiven.atlassian.net/browse/MA-3655
DOC PREVIEW
/docs/.