feat: add Logs - Azure Log Analytics module to ecosystem page#693
Conversation
Signed-off-by: Janaka Sandaruwan <janakasandaruwan1996@gmail.com>
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis 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. ChangesMarketplace plugin catalog entry
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
src/data/marketplace-plugins.json (1)
441-441: 💤 Low valueConsider adding an Azure-specific logo URL.
While an empty
logoUrlis 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
📒 Files selected for processing (1)
src/data/marketplace-plugins.json
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).