-
Notifications
You must be signed in to change notification settings - Fork 60
observability added #490
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
observability added #490
Conversation
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 🗂️ Base branches to auto review (7)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests
Tip 🧪 Early access (models): enabledWe are currently testing Sonnet 4.5 code review models, which should lead to better review quality. However, this model may result in higher noise levels in the review comments. Please disable the early access features if the noise level causes any inconvenience. Note:
Comment |
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
d8e1c6e
to
64e4837
Compare
64e4837
to
71e53fc
Compare
71e53fc
to
077b01d
Compare
d575a49
to
769797c
Compare
077b01d
to
4ad291f
Compare
4ad291f
to
5d6599f
Compare
e41f49d
to
1df0ef5
Compare
1df0ef5
to
db51a9a
Compare
Summary
Added a new Observability section to the UI with support for OpenTelemetry, Maxim, Datadog, and New Relic integrations. Also refactored HTTP middleware handling to improve type safety and error handling.
Changes
BifrostHTTPMiddleware
typeType of change
Affected areas
How to test
Test the new Observability UI and middleware changes:
Navigate to the new Observability page to verify the UI components render correctly.
Screenshots/Recordings
N/A
Breaking changes
Related issues
N/A
Security considerations
No significant security implications. The middleware refactoring improves request handling but doesn't change authentication or authorization logic.
Checklist
docs/contributing/README.md
and followed the guidelines