-
Notifications
You must be signed in to change notification settings - Fork 45
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
Adding this to KoolKits? #4
Comments
Hi @TomGranot, Will be happy to open a PR if you think it's a good idea. |
@edeNFed Please do! That would be awesome. If you can also add it to the README for the Go KoolKit with a quick explainer I'll appreciate that too:) We usually also ask maintainers to add a reference to the tool in their own READMEs - i.e. "To run this in k8s you can use a KoolKit" - would that be OK with you? I can open a PR here. |
Sure no problem, I'll let you know once I finished adding to KoolKit so we can update the README file. |
@edeNFed Dope. Talk soon! |
Hi @TomGranot, securityContext:
runAsUser: 0
capabilities:
add:
- SYS_PTRACE
privileged: true
volumeMounts:
- mountPath: /sys/kernel/debug
name: kernel-debug I could not find a flag in |
@edeNFed Interesting. To add what I know off the top of my head, If I understand correctly, you were attempting to target the latter - copying a pod and debugging it. I'm pretty sure that if you use that you indeed can't change the securityContext, so using 'node mode" seems like the right path to go. Let me know if I can help out with something! |
@edeNFed Any luck testing this out? |
@TomGranot sorry for the delay, I had a couple of busy days. |
@edeNFed NP! Just let me know if i can help. |
Hiya!
I've caught this repo from Observability News by Michael Hasuenblas - great stuff:)
I wonder, since this requires 0 code changes, whether it will be a good fit to bundle into our OSS project KoolKits.
We have docker debug images specifically geared towards debugging applications written in a specific language - you can check out the project here.
This might fit into our Go KoolKit.
Your comments much appreciated @edeNFed :)
The text was updated successfully, but these errors were encountered: