- Issue created by @kim.pepper
- Status changed to Needs work
over 1 year ago 2:57pm 4 April 2023 - π¦π²Armenia murz Yerevan, Armenia
I've implemented a dummy plugin here https://git.drupalcode.org/project/opentelemetry/-/blob/1.0.x/src/Plugin... that just checks the requirements for now.
The next step is to make it do the real work with auto-instrumentation. - First commit to issue fork.
- π³π±Netherlands euphoric_mv
Hello @Murz.
I am currently implementing auto-instrumentation on one of our projects, it works fine, so I would like to create a fork and push it next week. I will push it without tests, so you can have just a picture of how it works. Once we agree on an approach, I will add some tests.
- π¦π²Armenia murz Yerevan, Armenia
Good to know, thank you, will wait for your contribution to review! Wanted to do it by myself, but still very busy at work and have no time.
Hi @euphoric_mv, @Murz,
Can you please let me know the implementation of Auto instrumentation with drupal 10, php 8.1.
I have already enabled opentelemetry extension, what else is required for enabling auto instrumentation.
Thanks!
- π³π±Netherlands euphoric_mv
Hi @sudhanjs
Thank you for the reminder. I am working on a porting solution that I implemented in the project in this a contrib module. I will create PR and let you know guys to review it.
- π³π±Netherlands euphoric_mv
I'm trying to fork the project, but no success. Does anyone has the same problem?
- Status changed to Active
10 months ago 6:18am 22 January 2024 - π¦π²Armenia murz Yerevan, Armenia
In other issues, I see other folks have no problem with forking.
@euphoric_mv, could you please describe more details about your problems with forking? - π«π·France andypost
Used to hack a bit this auto-instrumentation but since OT 1.1 Fibers require to pass context...
So would be great to unify approach about sharing data between module's service and static context for auto-instrumentation.
There's workaround for Fibers in docs but having ZTS+ffi PHP only for tracing reasons IMO overkill
- π¦π²Armenia murz Yerevan, Armenia
By the way, I got an issue with Fibers after updating to the new OpenTelemetry libraries (1.1+), because they conflict with Drupal Fibers, used in the Renderer service.
So for now I just disabled them, and will investigate the issue later, here is the task: π Make OpenTelemetry work with FiberBoundContextStorage Active