-
Notifications
You must be signed in to change notification settings - Fork 810
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
Integrating Tracing Support for HAProxy in Instana #2782
Comments
Hi @capflam, Could you please help me on this? |
I don't know how Instana works, so it is hard for me to fully answer you. About the OpenTracing support. Because the project is fully dead since now 2 years, the HAProxy add-on will be removed, may be after a deprecation period, but independently on OpenTelemetry support. An OpenTelemetry add-on will be welcome if anyone is willing to work on it and to maintain it. I'm tempted to say that if it was not implemented in 2 years to replace the Opentracing add-on, it means there is not a strong traction from the community. But I may be wrong. Technically speaking, you must base your work on OpenTelemetry, definitely. But I don't know what implies an Instana tracing plugin. You must elaborate. But first of all, you should land the subject on the mailing-list because add-ons are only added if there are someone ready to use it. It is useless to have a long list of unused add-on. |
Of course, you must also check it is compatible with the HAproxy licence. You may check CONTRIBUTING file. |
The OpenTelemetry filter is under development and will be very similar to the current OpenTracing filter. However, it is not that simple (as it was not simple for OpenTracing either) because first I have to make a C wrapper for opentelemetry-cpp API. It will probably be usable in the 3.2 version of HAProxy. Also, even though OpenTelemetry has been in development for several years, it has not been a complete and stable C++ API for a long time, so I waited for it to finish. Regarding Instana, never worked with it, but I would repeat what @capflam wrote: "base it on OpenTelemetry". |
Miroslav mentioned below that he's currently working on an OpenTelemetry replacement for the OpenTracing filter since OpenTracing itself is no longer maintained nor supported: #2782 (comment) Given that he aims for 3.2, let's already settle on an upcoming deprecation of the filter for 3.3 with a removal for 3.5. This will leave time to finish the development and permit users to switch smoothly. At this point no warning is emitted (since the users have no alternative) but better mention this plan in the doc to make them aware of future changes.
Your Feature Request
Instana is planning to add tracing support to HAProxy and needs to confirm a few details before proceeding. We've seen in Git Hub that HAProxy is planning to integrate the OpenTelemetry (OTEL) support, so we'd like to confirm the current status of OTEL support.
Also we would like to understand if there's an opportunity for us to contribute our Instana tracing plugin, which is based on OpenTracing, while ensuring long-term support.
Could you please confirm which method would be preferable for you—OpenTracing or OpenTelemetry?
If the OTEL support is feasible, we'd like to know if we can contribute specific code changes, such as dropping Instana headers, adding resource attributes, infrastructure linking, OTel metrics.
Your feedback will help us to plan the best way to proceed.
What are you trying to do?
Planning to add tracing support to HAProxy in Instana.
Output of
haproxy -vv
The text was updated successfully, but these errors were encountered: