Skip to content

Conversation

@JAORMX
Copy link
Contributor

@JAORMX JAORMX commented Sep 9, 2025

  • Add comprehensive logging infrastructure documentation covering standard and audit logs
  • Document log format specifications with field definitions
  • Include configuration examples for operator and MCPServer resources
  • Add log collection strategies and enterprise tool integration (ELK, Splunk, Datadog)
  • Update sidebar navigation to include the new guide
  • Add contextual reference from deploy-operator-helm.md

Addresses: https://github.com/stacklok/toolhive-enterprise/issues/39

Copilot AI review requested due to automatic review settings September 9, 2025 08:37
@vercel
Copy link

vercel bot commented Sep 9, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
docs-website Ready Ready Preview Comment Sep 12, 2025 9:02am

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 adds comprehensive logging infrastructure documentation for ToolHive Kubernetes deployments, covering standard application logs and audit logs for security and compliance monitoring.

Key changes:

  • Comprehensive logging guide covering log formats, configuration, and enterprise tool integration
  • Audit logging specifications with detailed field definitions and event types
  • Log collection strategies using Kubernetes native tools and enterprise systems (ELK, Splunk, Datadog)

Reviewed Changes

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

File Description
sidebars.ts Add logging infrastructure guide to the Kubernetes guides section
docs/toolhive/guides-k8s/logging-infrastructure.md New comprehensive guide covering logging setup, configuration, parsing, and enterprise integrations
docs/toolhive/guides-k8s/deploy-operator-helm.md Add contextual reference to the new logging infrastructure guide

@danbarr
Copy link
Collaborator

danbarr commented Sep 9, 2025

@JAORMX Is this still a WIP? The build is failing because the next steps section links to nonexistent pages.

@JAORMX
Copy link
Contributor Author

JAORMX commented Sep 9, 2025

Ah, I can't see the logs of the build.

@danbarr
Copy link
Collaborator

danbarr commented Sep 9, 2025

Ah, I can't see the logs of the build.

Oh, right. The dev server (npm run start) just warns about broken links, but the production build fails out. You can run npm run build locally / in the dev container to see the same thing that Vercel is running, it'll tell you exactly what/where the broken links are.

@JAORMX
Copy link
Contributor Author

JAORMX commented Sep 9, 2025

Gotcha, I'll fix it tomorrow

- Add comprehensive logging infrastructure documentation covering standard and audit logs
- Document log format specifications with field definitions
- Include configuration examples for operator and MCPServer resources
- Add log collection strategies and enterprise tool integration (ELK, Splunk, Datadog)
- Update sidebar navigation to include the new guide
- Add contextual reference from deploy-operator-helm.md

Addresses: stacklok/toolhive-platform#39
Signed-off-by: Juan Antonio Osorio <[email protected]>
@JAORMX JAORMX merged commit 0ff10ec into main Sep 12, 2025
6 checks passed
@JAORMX JAORMX deleted the docs/k8s-logging-infrastructure branch September 12, 2025 10:02
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.

4 participants