-
Notifications
You must be signed in to change notification settings - Fork 140
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
Type 'Span' is missing the following properties from type 'Span': addLink, addLinks #1370
Comments
@gunzip Thank you for this catch. We'll update the appropriate OTel deps in the App Insights v3 project as they're out of sync with the dependency on the |
Thank you @JacksonWeber! Are you gonna align the azure/functions package as well? I see monitor-opentelemetry uses 4.x |
@gunzip v4 of Azure Functions is supported in the distro at the moment, but there's further work to do to also support in App Insights 3.x SDK. I can work on that and should be able to support it by the time of the next release. |
@gunzip the above PR should resolve your issue. Let me know and we can close this issue. |
Yes, solved now! |
It looks like there's a mismatch between the version of these packages:
@opentelemetry/api@^1.9.0
@opentelemetry/sdk-trace-base@^1.24.0
I get this error when building:
Type 'Span' is missing the following properties from type 'Span': addLink, addLinks
as the implementation here: https://github.com/open-telemetry/opentelemetry-js/blob/v1.24.0/packages/opentelemetry-sdk-trace-base/src/Span.ts#L57 misses addLink(s) methods, a breaking change as stated into the CHANGELOG:
https://github.com/open-telemetry/opentelemetry-js/blob/main/api/CHANGELOG.md#190
The text was updated successfully, but these errors were encountered: