Skip to content
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

Jaeger sidecar makes b2k fail #292

Closed
BartNetJS opened this issue Jul 11, 2023 · 4 comments
Closed

Jaeger sidecar makes b2k fail #292

BartNetJS opened this issue Jul 11, 2023 · 4 comments
Labels
bug Something isn't working

Comments

@BartNetJS
Copy link
Contributor

When I annotate a deployment with sidecar.jaegertracing.io/inject to add a jaeger sidecar to the pod, b2k stops working with the message
Error: Failed to validate the requirements to replicate resources locally: Failed to identify the container to use in pod '.....-5786c96cf8-csq96'.
Jaeger agent is working fine and b2k debugging session also without the jaeger agent

@BartNetJS BartNetJS added the bug Something isn't working label Jul 11, 2023
@hsubramanianaks
Copy link
Collaborator

@BartNetJS Thanks for raising this issue. Can you send all the logs from location $HOME/tmp/Bridge To Kubernetes ? Jaeger is not supported side car proxy, I think making a simple code change here might fix your issue. I will look further into once you send me the logs.

@BartNetJS
Copy link
Contributor Author

@hsubramanianaks hereby the log.
Are you gone make that change? Or will I contribute?
vscode-logs.zip

@hsubramanianaks
Copy link
Collaborator

@BartNetJS Thanks for the logs, looking at the logs yep as I suspected it needs code changes to support Jeager. We welcome contributions, if you have the time to make the changes it will be better. Thank you. Please let me know.

BartNetJS added a commit to BartNetJS/Bridge-To-Kubernetes that referenced this issue Jul 12, 2023
cxznmhdcxz pushed a commit that referenced this issue Jul 21, 2023
@hsubramanianaks
Copy link
Collaborator

hsubramanianaks commented Aug 21, 2023

This is released to production, please feel free to test and close this issue. Thank you for your contribution. @BartNetJS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants