-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
[service] Include component id/type in start error #11837
base: main
Are you sure you want to change the base?
[service] Include component id/type in start error #11837
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #11837 +/- ##
=======================================
Coverage 91.77% 91.77%
=======================================
Files 462 462
Lines 24886 24886
=======================================
Hits 22840 22840
Misses 1664 1664
Partials 382 382 ☔ View full report in Codecov by Sentry. |
Co-authored-by: Bogdan Drutu <[email protected]>
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
Description
Wraps the component ID and type into the component's start retuned error. If
telemetry::logs::level
is < error (e.g fatal), the service error message does not give information about which pipeline component failed:Current error message:
With these changes:
Link to tracking issue
#10426
Testing
Documentation