chore: Refactor observability setup. Migrate monitoring configuration…#37577
chore: Refactor observability setup. Migrate monitoring configuration…#37577subhrashisdas merged 1 commit intoreleasefrom
Conversation
…s from Appsmith to Monitoring service.
WalkthroughThe changes involve the deletion of several configuration files related to observability tools in a Docker setup. The removed files include configurations for Docker Compose, Grafana dashboards, data sources, and Prometheus settings, as well as a local Tempo configuration. These deletions indicate a significant alteration in the observability setup, potentially simplifying or changing the approach to monitoring and logging within the application. Changes
Suggested labels
Suggested reviewers
📜 Recent review detailsConfiguration used: .coderabbit.yaml 📒 Files selected for processing (7)
💤 Files with no reviewable changes (7)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
Description
Cleanup of Monitoring will be done in multiple phases to avoid any disruptions. The first PR is to remove the default monitoring Compose files.
Tip
Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team).
Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR.
Fixes #
Issue Numberor
Fixes
Issue URLWarning
If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.
Automation
/ok-to-test tags="@tag.Sanity"
🔍 Cypress test results
Tip
🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/11929368913
Commit: af90c5f
Cypress dashboard.
Tags:
@tag.SanitySpec:
Wed, 20 Nov 2024 08:55:56 UTC
Communication
Should the DevRel and Marketing teams inform users about this change?
Summary by CodeRabbit