-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[receiver/jaegerreceiver] Add logs when the respective server starts in jaegerreceiver #36961
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Saumyacodes-40 <saumyabshah90@gmail.com>
Signed-off-by: Saumyacodes-40 <saumyabshah90@gmail.com>
please include a sample output of running a collector with jaeger receiver enabled (with all its transports) to show logs in action |
I've added the same in the PR description. Should anything be changed? |
nothing looks odd to you in the output? Why do you need to repeat |
Signed-off-by: Saumyacodes-40 <saumyabshah90@gmail.com>
Apologies, I overlooked there and didnt identify that it's been updated/added by default. |
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.
please add a ./.chloggen/ entry
Signed-off-by: Saumyacodes-40 <saumyabshah90@gmail.com>
Signed-off-by: Saumyacodes-40 <saumyabshah90@gmail.com>
need to run |
Signed-off-by: Saumyacodes-40 <saumyabshah90@gmail.com>
Description
Link to tracking issue
Fixes jaegertracing/jaeger#6412
How was change tested?
Running collector with
jaegerreceiver
enabled with its all transportsOutput
config file used