-
Notifications
You must be signed in to change notification settings - Fork 5
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
Improve Tracing Configurations #104
Conversation
renaz6
commented
Feb 22, 2023
•
edited
edited
- Closes Make Tracing Options More Configurable #102
- Splitting up the tracing configuration changes so the user can decide exactly what gets traced and when
Codecov Report
@@ Coverage Diff @@
## main #104 +/- ##
==========================================
+ Coverage 73.46% 77.20% +3.73%
==========================================
Files 4 4
Lines 196 215 +19
==========================================
+ Hits 144 166 +22
+ Misses 46 44 -2
+ Partials 6 5 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm 🍻
config.go
Outdated
|
||
// NoParent decides if a root span should be initiated in the case where there is no existing parent | ||
// This value is ignored if ParentBased = "ignore" | ||
// "never" is the default value |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a value of always
also available?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, there is. I'll add it to the comment as well.
Kudos, SonarCloud Quality Gate passed! |