Skip to content

Commit d9af321

Browse files
authored
chore: remove the non-publishing packages from the npm workspace (open-telemetry#1938)
* chore: remove the non-publishing packages from the npm workspace Because they cause difficulties with updating `@opentelemetry/*` dependencies. This is "Option 3" from open-telemetry#1917 Refs: open-telemetry#1917 * undo open-telemetry#1917 option 1 work (adding non-publishing packages to release-please config) from open-telemetry#1928 * move express example (haven't adjusted build / docs yet) * express example updates to get it working * remove non-publishing packages from release-please manifest * move koa example * fix up the koa-example-related files * move the mongodb/examples files to the top-level * fixup the mongodb-example files * move mysql-example files * fixup the mysql-example files * move redis-example files to top level examples/ dir * fixup redis-example files * update examples README to no longer suggest moving to instruemtnation package dirs * regenerate package-lock.json to rm the '.../examples' dir entries 'npm install --package-lock-only' did not accomplish this. This regen was necessary because those vestigial entries caused surprising breakage in some 'npm install --no-save ...' commands such as TAV is doing. It broke TAV tests with [email protected]. * compile:examples is no longer a thing * fix for 'npm ci' failures imported from PR open-telemetry#1955 * regenerate package-lock.json 'undici-types', required by the version of @types/node used by the new instrumentation-perf-hooks, was lost in the merge from main. * add some more test output information to try to help debug flaky test * fix tweaks to the test * undo the instr-perf-hooks test assert tweaks, leaving that to a separate PR
1 parent 7d5cdb0 commit d9af321

File tree

95 files changed

+4935
-6666
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

95 files changed

+4935
-6666
lines changed

.github/workflows/unit-test.yml

-2
Original file line numberDiff line numberDiff line change
@@ -136,8 +136,6 @@ jobs:
136136
- name: Unit tests (Delta)
137137
if: ${{ !matrix.code-coverage }}
138138
run: npm run test:ci:changed
139-
- name: Build examples
140-
run: npm run compile:examples
141139
- name: Report Coverage
142140
if: ${{ matrix.code-coverage && !cancelled()}}
143141
uses: codecov/codecov-action@v3

.release-please-manifest.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
{"detectors/node/opentelemetry-resource-detector-alibaba-cloud":"0.28.6","detectors/node/opentelemetry-resource-detector-aws":"1.3.6","detectors/node/opentelemetry-resource-detector-azure":"0.2.4","detectors/node/opentelemetry-resource-detector-container":"0.3.6","detectors/node/opentelemetry-resource-detector-gcp":"0.29.6","detectors/node/opentelemetry-resource-detector-github":"0.28.1","detectors/node/opentelemetry-resource-detector-instana":"0.6.0","metapackages/auto-instrumentations-node":"0.41.1","metapackages/auto-instrumentations-web":"0.36.0","packages/opentelemetry-host-metrics":"0.35.0","packages/opentelemetry-id-generator-aws-xray":"1.2.1","packages/opentelemetry-propagation-utils":"0.30.6","packages/opentelemetry-redis-common":"0.36.1","packages/opentelemetry-sampler-aws-xray":"0.34.0","packages/opentelemetry-sql-common":"0.40.0","packages/opentelemetry-test-utils":"0.36.0","plugins/node/instrumentation-amqplib":"0.34.0","plugins/node/instrumentation-cucumber":"0.3.0","plugins/node/instrumentation-dataloader":"0.6.0","plugins/node/instrumentation-fs":"0.9.0","plugins/node/instrumentation-lru-memoizer":"0.34.0","plugins/node/instrumentation-mongoose":"0.35.0","plugins/node/instrumentation-perf-hooks": "0.1.0","plugins/node/instrumentation-socket.io":"0.36.0","plugins/node/instrumentation-tedious":"0.7.0","plugins/node/opentelemetry-instrumentation-aws-lambda":"0.38.0","plugins/node/opentelemetry-instrumentation-aws-sdk":"0.38.1","plugins/node/opentelemetry-instrumentation-bunyan":"0.35.0","plugins/node/opentelemetry-instrumentation-bunyan/examples":"0.1.0","plugins/node/opentelemetry-instrumentation-cassandra":"0.35.0","plugins/node/opentelemetry-instrumentation-connect":"0.33.0","plugins/node/opentelemetry-instrumentation-dns":"0.33.0","plugins/node/opentelemetry-instrumentation-express":"0.35.0","plugins/node/opentelemetry-instrumentation-express/examples":"0.1.0","plugins/node/opentelemetry-instrumentation-fastify":"0.33.0","plugins/node/opentelemetry-instrumentation-generic-pool":"0.33.0","plugins/node/opentelemetry-instrumentation-graphql":"0.37.0","plugins/node/opentelemetry-instrumentation-hapi":"0.34.0","plugins/node/opentelemetry-instrumentation-ioredis":"0.37.0","plugins/node/opentelemetry-instrumentation-knex":"0.33.0","plugins/node/opentelemetry-instrumentation-koa":"0.37.0","plugins/node/opentelemetry-instrumentation-koa/examples":"0.1.0","plugins/node/opentelemetry-instrumentation-memcached":"0.33.0","plugins/node/opentelemetry-instrumentation-mongodb":"0.39.0","plugins/node/opentelemetry-instrumentation-mongodb/examples":"0.1.0","plugins/node/opentelemetry-instrumentation-mysql":"0.35.0","plugins/node/opentelemetry-instrumentation-mysql/examples":"0.1.0","plugins/node/opentelemetry-instrumentation-mysql2":"0.35.0","plugins/node/opentelemetry-instrumentation-nestjs-core":"0.34.0","plugins/node/opentelemetry-instrumentation-net":"0.33.0","plugins/node/opentelemetry-instrumentation-pg":"0.38.0","plugins/node/opentelemetry-instrumentation-pino":"0.35.0","plugins/node/opentelemetry-instrumentation-redis":"0.36.0","plugins/node/opentelemetry-instrumentation-redis/examples":"0.1.0","plugins/node/opentelemetry-instrumentation-redis-4":"0.36.0","plugins/node/opentelemetry-instrumentation-restify":"0.35.0","plugins/node/opentelemetry-instrumentation-router":"0.34.0","plugins/node/opentelemetry-instrumentation-winston":"0.34.0","plugins/web/opentelemetry-instrumentation-document-load":"0.35.0","plugins/web/opentelemetry-instrumentation-long-task":"0.35.0","plugins/web/opentelemetry-instrumentation-user-interaction":"0.35.0","plugins/web/opentelemetry-plugin-react-load":"0.30.0","propagators/opentelemetry-propagator-aws-xray":"1.3.1","propagators/opentelemetry-propagator-grpc-census-binary":"0.27.1","propagators/opentelemetry-propagator-instana":"0.3.1","propagators/opentelemetry-propagator-ot-trace":"0.27.1"}
1+
{"detectors/node/opentelemetry-resource-detector-alibaba-cloud":"0.28.6","detectors/node/opentelemetry-resource-detector-aws":"1.3.6","detectors/node/opentelemetry-resource-detector-azure":"0.2.4","detectors/node/opentelemetry-resource-detector-container":"0.3.6","detectors/node/opentelemetry-resource-detector-gcp":"0.29.6","detectors/node/opentelemetry-resource-detector-github":"0.28.1","detectors/node/opentelemetry-resource-detector-instana":"0.6.0","metapackages/auto-instrumentations-node":"0.41.1","metapackages/auto-instrumentations-web":"0.36.0","packages/opentelemetry-host-metrics":"0.35.0","packages/opentelemetry-id-generator-aws-xray":"1.2.1","packages/opentelemetry-propagation-utils":"0.30.6","packages/opentelemetry-redis-common":"0.36.1","packages/opentelemetry-sql-common":"0.40.0","packages/opentelemetry-test-utils":"0.36.0","plugins/node/instrumentation-amqplib":"0.34.0","plugins/node/instrumentation-cucumber":"0.3.0","plugins/node/instrumentation-dataloader":"0.6.0","plugins/node/instrumentation-fs":"0.9.0","plugins/node/instrumentation-lru-memoizer":"0.34.0","plugins/node/instrumentation-mongoose":"0.35.0","plugins/node/instrumentation-perf-hooks": "0.1.0","plugins/node/instrumentation-socket.io":"0.36.0","plugins/node/instrumentation-tedious":"0.7.0","plugins/node/opentelemetry-instrumentation-aws-lambda":"0.38.0","plugins/node/opentelemetry-instrumentation-aws-sdk":"0.38.1","plugins/node/opentelemetry-instrumentation-bunyan":"0.35.0","plugins/node/opentelemetry-instrumentation-cassandra":"0.35.0","plugins/node/opentelemetry-instrumentation-connect":"0.33.0","plugins/node/opentelemetry-instrumentation-dns":"0.33.0","plugins/node/opentelemetry-instrumentation-express":"0.35.0","plugins/node/opentelemetry-instrumentation-fastify":"0.33.0","plugins/node/opentelemetry-instrumentation-generic-pool":"0.33.0","plugins/node/opentelemetry-instrumentation-graphql":"0.37.0","plugins/node/opentelemetry-instrumentation-hapi":"0.34.0","plugins/node/opentelemetry-instrumentation-ioredis":"0.37.0","plugins/node/opentelemetry-instrumentation-knex":"0.33.0","plugins/node/opentelemetry-instrumentation-koa":"0.37.0","plugins/node/opentelemetry-instrumentation-memcached":"0.33.0","plugins/node/opentelemetry-instrumentation-mongodb":"0.39.0","plugins/node/opentelemetry-instrumentation-mysql":"0.35.0","plugins/node/opentelemetry-instrumentation-mysql2":"0.35.0","plugins/node/opentelemetry-instrumentation-nestjs-core":"0.34.0","plugins/node/opentelemetry-instrumentation-net":"0.33.0","plugins/node/opentelemetry-instrumentation-pg":"0.38.0","plugins/node/opentelemetry-instrumentation-pino":"0.35.0","plugins/node/opentelemetry-instrumentation-redis":"0.36.0","plugins/node/opentelemetry-instrumentation-redis-4":"0.36.0","plugins/node/opentelemetry-instrumentation-restify":"0.35.0","plugins/node/opentelemetry-instrumentation-router":"0.34.0","plugins/node/opentelemetry-instrumentation-winston":"0.34.0","plugins/web/opentelemetry-instrumentation-document-load":"0.35.0","plugins/web/opentelemetry-instrumentation-long-task":"0.35.0","plugins/web/opentelemetry-instrumentation-user-interaction":"0.35.0","plugins/web/opentelemetry-plugin-react-load":"0.30.0","propagators/opentelemetry-propagator-aws-xray":"1.3.1","propagators/opentelemetry-propagator-grpc-census-binary":"0.27.1","propagators/opentelemetry-propagator-instana":"0.3.1","propagators/opentelemetry-propagator-ot-trace":"0.27.1"}

examples/README.md

+3-38

examples/bunyan/.npmrc

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
package-lock=false

examples/express/.npmrc

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
package-lock=false

examples/express/README.md

+71

plugins/node/opentelemetry-instrumentation-express/examples/package.json examples/express/package.json

+3-4
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,12 @@
88
"zipkin:client": "cross-env EXPORTER=zipkin ts-node src/client.ts",
99
"jaeger:server": "cross-env EXPORTER=jaeger ts-node src/server.ts",
1010
"jaeger:client": "cross-env EXPORTER=jaeger ts-node src/client.ts",
11-
"compile": "tsc -p .",
12-
"setup": "cd ../../../../ && npm ci && cd plugins/node/opentelemetry-instrumentation-express && npm run compile && cd examples && npm run compile"
11+
"compile": "tsc -p ."
1312
},
1413
"repository": {
1514
"type": "git",
1615
"url": "git+ssh://[email protected]/open-telemetry/opentelemetry-js-contrib.git",
17-
"directory": "plugins/node/opentelemetry-instrumentation-express"
16+
"directory": "examples/express"
1817
},
1918
"keywords": [
2019
"opentelemetry",
@@ -29,7 +28,7 @@
2928
"bugs": {
3029
"url": "https://github.com/open-telemetry/opentelemetry-js-contrib/issues"
3130
},
32-
"homepage": "https://github.com/open-telemetry/opentelemetry-js-contrib/tree/main/plugins/node/opentelemetry-instrumentation-express/examples#readme",
31+
"homepage": "https://github.com/open-telemetry/opentelemetry-js-contrib/tree/main/examples/express#readme",
3332
"dependencies": {
3433
"@opentelemetry/api": "^1.3.0",
3534
"@opentelemetry/exporter-jaeger": "^1.18.1",

plugins/node/opentelemetry-instrumentation-koa/examples/tsconfig.json examples/express/tsconfig.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"extends": "../tsconfig.json",
2+
"extends": "../../tsconfig.base",
33
"compilerOptions": {
44
"noEmit": true,
55
"rootDir": ".",

examples/koa/.npmrc

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
package-lock=false

plugins/node/opentelemetry-instrumentation-koa/examples/README.md examples/koa/README.md

+2-2

plugins/node/opentelemetry-instrumentation-koa/examples/package.json examples/koa/package.json

+3-3
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
"private": true,
44
"version": "0.1.0",
55
"description": "Example of Koa and @koa/router integration with OpenTelemetry",
6-
"main": "index.js",
76
"scripts": {
87
"zipkin:server": "cross-env EXPORTER=zipkin ts-node src/server.ts",
98
"zipkin:client": "cross-env EXPORTER=zipkin ts-node src/client.ts",
@@ -13,7 +12,8 @@
1312
},
1413
"repository": {
1514
"type": "git",
16-
"url": "git+ssh://[email protected]/open-telemetry/opentelemetry-js-contrib.git"
15+
"url": "git+ssh://[email protected]/open-telemetry/opentelemetry-js-contrib.git",
16+
"directory": "examples/koa"
1717
},
1818
"keywords": [
1919
"opentelemetry",
@@ -42,7 +42,7 @@
4242
"axios": "^1.6.0",
4343
"koa": "^2.13.0"
4444
},
45-
"homepage": "https://github.com/open-telemetry/opentelemetry-js-contrib#readme",
45+
"homepage": "https://github.com/open-telemetry/opentelemetry-js-contrib/tree/main/examples/koa#readme",
4646
"devDependencies": {
4747
"cross-env": "^7.0.0",
4848
"ts-node": "^10.6.0",

plugins/node/opentelemetry-instrumentation-mongodb/examples/tsconfig.json examples/koa/tsconfig.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"extends": "../tsconfig.json",
2+
"extends": "../../tsconfig.base",
33
"compilerOptions": {
44
"noEmit": true,
55
"rootDir": ".",

examples/mongodb/.npmrc

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
package-lock=false

plugins/node/opentelemetry-instrumentation-mongodb/examples/README.md examples/mongodb/README.md

+5-5

plugins/node/opentelemetry-instrumentation-express/examples/tsconfig.json examples/mongodb/tsconfig.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"extends": "../tsconfig.json",
2+
"extends": "../../tsconfig.base",
33
"compilerOptions": {
44
"noEmit": true,
55
"rootDir": ".",

examples/mysql/.npmrc

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
package-lock=false

plugins/node/opentelemetry-instrumentation-mysql/examples/README.md examples/mysql/README.md

+3-3

plugins/node/opentelemetry-instrumentation-mysql/examples/package.json examples/mysql/package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@
4242
"@opentelemetry/exporter-metrics-otlp-grpc": "^0.48.0",
4343
"mysql": "^2.18.1"
4444
},
45-
"homepage": "https://github.com/open-telemetry/opentelemetry-js#readme",
45+
"homepage": "https://github.com/open-telemetry/opentelemetry-js/tree/main/examples/mysql#readme",
4646
"devDependencies": {
4747
"@types/mysql": "^2.15.21",
4848
"cross-env": "^7.0.0",

plugins/node/opentelemetry-instrumentation-mysql/examples/src/server.ts examples/mysql/src/server.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ function startServer(port: number | undefined) {
5050
function handleRequest(request: any, response: any) {
5151
const currentSpan = api.trace.getSpan(api.context.active())
5252
// display traceid in the terminal
53-
const traceId = currentSpan?.spanContext();
53+
const traceId = currentSpan?.spanContext().traceId;
5454
console.log(`traceid: ${traceId}`);
5555
console.log(`Jaeger URL: http://localhost:16686/trace/${traceId}`);
5656
console.log(`Zipkin URL: http://localhost:9411/zipkin/traces/${traceId}`);

plugins/node/opentelemetry-instrumentation-mysql/examples/tsconfig.json examples/mysql/tsconfig.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"extends": "../tsconfig.json",
2+
"extends": "../../tsconfig.base",
33
"compilerOptions": {
44
"noEmit": true,
55
"rootDir": ".",

examples/redis/.npmrc

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
package-lock=false

0 commit comments

Comments
 (0)