Skip to content

adds Azure Logs docs. #189

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jonhadfield
Copy link
Contributor

Adds docs for wtfutil/wtf#1799.

@Copilot Copilot AI review requested due to automatic review settings July 14, 2025 19:53
Copy link

@Copilot 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

Adds documentation for a new Azure Logs module and registers it in the site navigation.

  • Introduces an “Azure Logs” entry in the MkDocs navigation
  • Provides a configuration example, sample Kusto query file, and attribute reference for the Azure Logs widget

Reviewed Changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated 1 comment.

File Description
mkdocs.yml Add “Azure Logs” entry to the navigation
docs/modules/azure_logs.md Create initial docs for the Azure Logs module
Comments suppressed due to low confidence (2)

docs/modules/azure_logs.md:3

  • The service is called 'Azure Log Analytics' without extra dashes or spaces; update to 'Displays results from an Azure Log Analytics Workspace query.'
Displays results from an Azure - Log Analytics Workspace query.

docs/modules/azure_logs.md:38

  • Only 'title' and 'queryFile' are documented. Consider adding descriptions for 'enabled', 'position', and 'refreshInterval' to fully cover the module’s configuration options.
## Attributes

```yaml
azure_subscription_id: 6a258992-8b46-4657-ae2f-1384f2c94af1
azure_workspace_id: 0f244ff4-ddd1-468e-8587-933a54a1ef07
columns: ["method", "status", "count"]
Copy link
Preview

Copilot AI Jul 14, 2025

Choose a reason for hiding this comment

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

These column names don’t match the fields output by the sample query (RequestCount, HTTPMethod, StatusCode); update to ["HTTPMethod", "StatusCode", "RequestCount"] or adjust the query accordingly.

Suggested change
columns: ["method", "status", "count"]
columns: ["HTTPMethod", "StatusCode", "RequestCount"]

Copilot uses AI. Check for mistakes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant