Skip to content
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

Remove incorrect monitoring information and delete rogue monitoring page #52145

Merged
merged 2 commits into from
Jan 25, 2025

Conversation

ian-at-airbyte
Copy link
Contributor

What

Resolves #11478 by removing out-of-date monitoring information. This caused all kinds of problems for the community because it doesn't work.

How

  • Remove out-of-date/incorrect Datadog and OTEL informaton. Clarify that we use Datadog for Cloud monitoring only.
  • Tell enterprise customers to let us know if they want Datadog for their own deployments.
  • Remove rogue empty monitoring page that was indexed in search but returned a blank page.

Review guide

  • /docs/operator-guides/collecting-metrics.md

User Impact

Avoid the false implication that Community members can do this.

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

Copy link

vercel bot commented Jan 25, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 25, 2025 0:58am

@ian-at-airbyte ian-at-airbyte merged commit d457845 into master Jan 25, 2025
25 checks passed
@ian-at-airbyte ian-at-airbyte deleted the #11478-monitoring-docs-placeholder branch January 25, 2025 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants