-
Notifications
You must be signed in to change notification settings - Fork 39
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
Instrument with OpenTelemetry #354
Comments
Hello @agracey , Totally agree with you this is needed, i am starting to work on implementing this in rancher code using the opentelemtry SDK, however i see that opentelemetry packages are already there in go.mod with very old versions and can't find where they are use ? Also when i tried to update them so i can start implementing the SDK lot of other dependencies pkg start showing errors. Any idea how we can make clean implementation for this ? |
Any updates on this? |
Hey @krumware 👋🏼 |
Sounds good thanks for the update. It's starting to come up in convos with Prime customers, so just staying out ahead. Thanks! |
It may be worth adding OpenTelemetry instrumentation to the operator (and cli?) for a few reasons:
Note: I would not expect that we force everyone to setup jaeger or prometheus. But being able to add the right flags to get the data published would be useful when appropriate.
The text was updated successfully, but these errors were encountered: