-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Fluentbit opentelemetry output plugin does not format fields correctly #8552
Comments
Can this please be triaged |
Hey! I've been keeping up with your work here and I just wanted to note that I'm not a Fluent Bit maintainer. I'm primarily a user of Fluent Bit and a light contributor. I'm happy to compile and test code though to help where I can. Hopefully a maintainer can take a look at this and review your linked pull request soon. |
@sudomateo Thank you, Please take a look at my pull and test it out if you can. I was able to test and get it working but if you can also test in your scenrio that would be great. I have added support for spanid, traceid, severitytext, and severitynumber. So those are the ones you can try. I havnt yet decided how we would want to handle adding attributes so if you have suggestions then that would be great. |
This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 5 days. Maintainers can add the |
This issue was closed because it has been stalled for 5 days with no activity. |
This is a follow up to issue 8359. That ticket was closed however my main issue was NOT fixed. The TOPLEVEL traceid and spanid fields are still not being set. If there is another PR or ticket for this work then please post it.
I have a PR open to resolve this
#8644
The text was updated successfully, but these errors were encountered: