Skip to content

feat: add Logs - Azure Log Analytics module to ecosystem page#693

Merged
JanakaSandaruwan merged 1 commit into
openchoreo:mainfrom
JanakaSandaruwan:feat/azure-loganalytics-marketplace
Jun 9, 2026
Merged

feat: add Logs - Azure Log Analytics module to ecosystem page#693
JanakaSandaruwan merged 1 commit into
openchoreo:mainfrom
JanakaSandaruwan:feat/azure-loganalytics-marketplace

Conversation

@JanakaSandaruwan

Copy link
Copy Markdown
Contributor

Adds the Observability Logs Module for Azure Log Analytics to the ecosystem marketplace, alongside the existing AWS CloudWatch and OpenSearch logs entries.

Source module: https://github.com/openchoreo/community-modules/tree/main/observability-logs-azure-loganalytics (PR #132).

Signed-off-by: Janaka Sandaruwan <janakasandaruwan1996@gmail.com>
@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Summary by CodeRabbit

  • New Features
    • Added Azure Log Analytics plugin to the marketplace, enabling seamless integration for logs aggregation, analysis, and monitoring.

Walkthrough

This PR adds a new marketplace plugin entry for "Logs - Azure Log Analytics" to the plugin catalog. The entry includes the plugin's category, tags, description, logo URL, and module source URL, positioned alongside existing logging provider plugins.

Changes

Marketplace plugin catalog entry

Layer / File(s) Summary
Azure Log Analytics plugin entry
src/data/marketplace-plugins.json
A new logs-azure-loganalytics plugin entry is added to the marketplace catalog with category, tags, description, metadata flags (released: true, default: false), and source URL pointing to the Azure Log Analytics module.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • LakshanSS
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is incomplete. It lacks the required 'Purpose' section structure and checklist items specified in the repository template. Restructure the description to follow the template: add a 'Purpose' section, 'Related Issues' section, and a complete checklist with checkboxes marking completed items.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The pull request title clearly and specifically summarizes the main change: adding a new Logs - Azure Log Analytics module to the ecosystem page.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
src/data/marketplace-plugins.json (1)

441-441: 💤 Low value

Consider adding an Azure-specific logo URL.

While an empty logoUrl is acceptable and will fall back to the default OpenChoreo logo, Azure provides official brand assets that could improve visual recognition. The logs-cloudwatch entry also has an empty logoUrl, but consider adding one for better UX.

For reference, Azure Monitor and Log Analytics logos are available from Microsoft's official resources.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/data/marketplace-plugins.json` at line 441, The marketplace entry has an
empty "logoUrl" value; replace the empty string for the Azure-specific plugin's
"logoUrl" with a stable Azure Monitor / Log Analytics asset URL (use Microsoft's
official brand asset link) so the plugin shows the proper icon, and apply the
same change to the "logoUrl" for the "logs-cloudwatch" entry if needed; locate
the JSON objects containing the "logoUrl" property for these plugin entries and
update their values to the chosen Azure logo URL.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@src/data/marketplace-plugins.json`:
- Line 441: The marketplace entry has an empty "logoUrl" value; replace the
empty string for the Azure-specific plugin's "logoUrl" with a stable Azure
Monitor / Log Analytics asset URL (use Microsoft's official brand asset link) so
the plugin shows the proper icon, and apply the same change to the "logoUrl" for
the "logs-cloudwatch" entry if needed; locate the JSON objects containing the
"logoUrl" property for these plugin entries and update their values to the
chosen Azure logo URL.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d68a117e-5aad-499d-8750-c68fe5244473

📥 Commits

Reviewing files that changed from the base of the PR and between fab868c and afaa9b1.

📒 Files selected for processing (1)
  • src/data/marketplace-plugins.json

@LakshanSS LakshanSS left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@JanakaSandaruwan JanakaSandaruwan merged commit 0540cef into openchoreo:main Jun 9, 2026
3 checks passed
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