search

appsignal / appsignal-nodejs

🟩 AppSignal for Node.js
https://www.appsignal.com/nodejs/
MIT License
28 stars 9 forks source link

Update OpenTelemetry package dependencies #1070

Closed tombruijn closed 2 months ago

tombruijn commented 2 months ago

Update OpenTelemetry package dependencies

Update the OpenTelemetry dependencies to fix issue #1068, about the types package for ioredis not being present anymore. I've updated everything up to OpenTelemetry API 1.8. Version 1.9 is also out, but the prisma package is not compatible with that at the moment.

Lock all dependencies to a minor version range so that it hopefully doesn't install different versions of the OpenTelemetry API package, which breaks things.

Related commit with more information: 32c499fa0c07535f45c940df1f3b8ab3aa3822b8 Closes #1068

Disable Client for probes test

If it were to try and export any metrics it would throw this error:

/home/semaphore/appsignal-nodejs/node_modules/@opentelemetry/sdk-metrics/build/src/export/PeriodicExportingMetricReader.js:76
                throw new Error(`PeriodicExportingMetricReader: metrics export failed (error ${result.error})`);
                      ^
Error: PeriodicExportingMetricReader: metrics export failed (error Error: connect ECONNREFUSED 127.0.0.1:8099)
    at doExport (/home/semaphore/appsignal-nodejs/node_modules/@opentelemetry/sdk-metrics/src/export/PeriodicExportingMetricReader.ts:133:15)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)