You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
New users trying out opentelemetry with OTLP exporting are frequently tripped up by not having a protobuf implementation installed, eg #1059 (comment) , #954 (comment) and #1045 (comment)
In #949 and #958 we made the protobuf implementation optional, the idea being that the user could choose their implementation but had to provide one.
add both to exporter-otlp's suggest section. People might notice this, but also might not.
add a post-install composer script to exporter-otlp which will complain if no implementation available
update google-protobuf native package to provide ext-protobuf adding provide to php composer.json protocolbuffers/protobuf#13141 - with this we can require ext-protobuf and have it provided by either native or extension. Getting patches into a released version of protobuf is a long game, though. (edit: is now released in 3.24.0)
require the native implementation and suggest the extension
This is what we originally had, and it has the benefit of working out of the box. If you install the extension as well, then it takes priority and will be used.
The text was updated successfully, but these errors were encountered:
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
New users trying out opentelemetry with OTLP exporting are frequently tripped up by not having a protobuf implementation installed, eg #1059 (comment) , #954 (comment) and #1045 (comment)
In #949 and #958 we made the protobuf implementation optional, the idea being that the user could choose their implementation but had to provide one.
Possible solutions
It's mentioned in https://github.com/open-telemetry/opentelemetry-php/tree/main/src/Contrib/Otlp#protobuf-runtime-library
exporter-otlp
'ssuggest
section. People might notice this, but also might not.adding provide to php composer.json protocolbuffers/protobuf#13141 - with this we can require
ext-protobuf
and have it provided by either native or extension. Getting patches into a released version of protobuf is a long game, though. (edit: is now released in 3.24.0)This is what we originally had, and it has the benefit of working out of the box. If you install the extension as well, then it takes priority and will be used.
The text was updated successfully, but these errors were encountered: