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

Reverted OpenTelemetry integration #766

Merged
merged 1 commit into from
Feb 17, 2025

Conversation

danyi1212
Copy link
Collaborator

@danyi1212 danyi1212 commented Feb 17, 2025

Revert OpenTelemetry Integration

Why this revert?

Since introducing OpenTelemetry support in v0.7.16, we’ve encountered multiple issues that have impacted both OPAL and its users. These include:

  • Bugs in trace collection, leading to incomplete or missing telemetry data.
  • Integration difficulties, especially in Alpine Linux environments.
  • Documentation gaps, making it harder for users to correctly configure and use OpenTelemetry with OPAL.
  • Unexpected stability issues, affecting both open-source users and Permit’s production environment.

These issues have significantly slowed down development and introduced challenges that we weren’t able to fully anticipate before merging the feature.

What’s Next?

  • OpenTelemetry will remain available in OPAL v0.7.x, where we’ll continue to apply patches and fixes.
  • Starting from OPAL v0.8.x, OpenTelemetry will be removed while we work on refining the implementation.
  • Once we’ve improved stability, documentation, and integration, we plan to reintroduce OpenTelemetry in a future release.

We appreciate everyone’s patience as we work through this, and we’re committed to bringing OpenTelemetry back in a way that’s more reliable and user-friendly. If you have feedback or suggestions, we’d love to hear from you!

@danyi1212 danyi1212 requested review from omer9564 and orweis February 17, 2025 10:42
@danyi1212 danyi1212 self-assigned this Feb 17, 2025
Copy link

netlify bot commented Feb 17, 2025

Deploy Preview for opal-docs ready!

Name Link
🔨 Latest commit 6eb0ad7
🔍 Latest deploy log https://app.netlify.com/sites/opal-docs/deploys/67b3128ee61fb50008b77c1a
😎 Deploy Preview https://deploy-preview-766--opal-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@danyi1212 danyi1212 merged commit bde6df7 into master Feb 17, 2025
11 checks passed
@danyi1212 danyi1212 deleted the dan/per-11869-revert-opentelemetry-from-opal branch February 17, 2025 11:18
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