-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Debug images with delve and other base image #2493
Comments
can I take this up ? |
It's yours! |
@vkartik97 do you need help in moving forward with this issue? |
@aebirim wants to work on this one as well. For some reason, I can't assign this to you, but it's yours. |
@jpkrohling is delve needed to be in the image for this? |
Delve needs to be there, and the Jaeger binaries in the image should not be optimized, so that Delve can extract name and line information. |
Requirement - what kind of business use case are you trying to solve?
We discussed having a different base image for Jaeger, for better troubleshooting, but never moved forward, mostly because the maintenance burden of having two very similar images wasn't worth it. But @bjung82 had a great question, on whether we have a debug image with delve enabled, for remote debugging.
This sounds like a great idea, and given the recent issues with the agent, sounds like something that would really help users.
Problem - what in Jaeger blocks you from solving the requirement?
Sometimes, observability is not enough ;-)
Proposal - what do you suggest to solve the problem or improve the existing situation?
This would require a new Dockerfile for all components, new targets in the Makefile.
Context: #2465, and would probably also be helpful for #2452
The text was updated successfully, but these errors were encountered: