Skip to content

Commit

Permalink
Merge branch 'main' into zipkin-env-vars
Browse files Browse the repository at this point in the history
  • Loading branch information
vmarchaud authored Apr 13, 2021
2 parents 1dee5c7 + 4ef22ca commit be2e2b8
Show file tree
Hide file tree
Showing 48 changed files with 372 additions and 254 deletions.
6 changes: 2 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -245,12 +245,10 @@ To request automatic tracing support for a module not on this list, please [file

## Upgrade guidelines

### 0.19.0 to 1.0.0-rc.0
### 0.18.x to 0.19.0

- All plugins have been removed in favor of instrumentations.

### 0.18.0 to 0.19.0


- The `@opentelemetry/propagator-b3` package previously exported three propagators: `B3Propagator`,`B3SinglePropagator`, and `B3MultiPropagator`, but now only exports the `B3Propagator`. It extracts b3 context in single and multi-header encodings, and injects context using the single-header encoding by default, but can be configured to inject context using the multi-header endcoding during construction: `new B3Propagator({ injectEncoding: B3InjectEncoding.MULTI_HEADER })`. If you were previously using the `B3SinglePropagator` or `B3MultiPropagator` directly, you should update your code to use the `B3Propagator` with the appropriate configuration. See the [readme](./packages/opentelemetry-propagator-b3/readme.md) for full details and usage.

### 0.17.0 to 0.18.0
Expand Down
6 changes: 3 additions & 3 deletions backwards-compatability/node10/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,12 @@
"test:backcompat": "tsc --noEmit index.ts"
},
"dependencies": {
"@opentelemetry/sdk-node": "^0.18.2",
"@opentelemetry/tracing": "^0.18.2"
"@opentelemetry/sdk-node": "0.18.2",
"@opentelemetry/tracing": "0.18.2"
},
"devDependencies": {
"@types/node": "10.17.56",
"typescript": "4.2.3"
"typescript": "4.2.4"
},
"author": "OpenTelemetry Authors",
"license": "Apache-2.0"
Expand Down
6 changes: 3 additions & 3 deletions backwards-compatability/node12/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,12 @@
"test:backcompat": "tsc --noEmit index.ts"
},
"dependencies": {
"@opentelemetry/sdk-node": "^0.18.2",
"@opentelemetry/tracing": "^0.18.2"
"@opentelemetry/sdk-node": "0.18.2",
"@opentelemetry/tracing": "0.18.2"
},
"devDependencies": {
"@types/node": "12.20.7",
"typescript": "4.2.3"
"typescript": "4.2.4"
},
"author": "OpenTelemetry Authors",
"license": "Apache-2.0"
Expand Down
6 changes: 3 additions & 3 deletions backwards-compatability/node8/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,12 @@
"test:backcompat": "tsc --noEmit index.ts"
},
"dependencies": {
"@opentelemetry/sdk-node": "^0.18.2",
"@opentelemetry/tracing": "^0.18.2"
"@opentelemetry/sdk-node": "0.18.2",
"@opentelemetry/tracing": "0.18.2"
},
"devDependencies": {
"@types/node": "8.10.66",
"typescript": "4.2.3"
"typescript": "4.2.4"
},
"author": "OpenTelemetry Authors",
"license": "Apache-2.0"
Expand Down
2 changes: 1 addition & 1 deletion examples/grpc-js/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
"url": "https://github.com/open-telemetry/opentelemetry-js/issues"
},
"dependencies": {
"@grpc/grpc-js": "^1.0.5",
"@grpc/grpc-js": "^1.2.12",
"@opentelemetry/api": "^1.0.0-rc.0",
"@opentelemetry/exporter-jaeger": "^0.18.2",
"@opentelemetry/exporter-zipkin": "^0.18.2",
Expand Down
8 changes: 4 additions & 4 deletions integration-tests/propagation-validation-server/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,16 +11,16 @@
"compile": "tsc --build"
},
"dependencies": {
"@opentelemetry/context-async-hooks": "^0.18.2",
"@opentelemetry/core": "^0.18.2",
"@opentelemetry/tracing": "^0.18.2",
"@opentelemetry/context-async-hooks": "0.18.2",
"@opentelemetry/core": "0.18.2",
"@opentelemetry/tracing": "0.18.2",
"axios": "0.21.1",
"body-parser": "1.19.0",
"express": "4.17.1"
},
"devDependencies": {
"@opentelemetry/api": "^1.0.0-rc.0",
"typescript": "4.2.3"
"typescript": "4.2.4"
},
"peerDependencies": {
"@opentelemetry/api": "^1.0.0-rc.0"
Expand Down
14 changes: 7 additions & 7 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -45,13 +45,13 @@
"author": "OpenTelemetry Authors",
"license": "Apache-2.0",
"devDependencies": {
"@commitlint/cli": "12.0.1",
"@commitlint/config-conventional": "12.0.1",
"@typescript-eslint/eslint-plugin": "4.20.0",
"@typescript-eslint/parser": "4.20.0",
"@commitlint/cli": "12.1.1",
"@commitlint/config-conventional": "12.1.1",
"@typescript-eslint/eslint-plugin": "4.22.0",
"@typescript-eslint/parser": "4.22.0",
"beautify-benchmark": "0.2.4",
"benchmark": "2.1.4",
"eslint": "7.23.0",
"eslint": "7.24.0",
"eslint-config-airbnb-base": "14.2.1",
"eslint-plugin-header": "3.1.1",
"eslint-plugin-import": "2.22.1",
Expand All @@ -61,8 +61,8 @@
"lerna": "3.22.1",
"lerna-changelog": "1.0.1",
"markdownlint-cli": "0.27.1",
"typescript": "4.2.3",
"update-ts-references": "2.0.0"
"typescript": "4.2.4",
"update-ts-references": "2.3.0"
},
"husky": {
"hooks": {
Expand Down
2 changes: 1 addition & 1 deletion packages/opentelemetry-api-metrics/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@
"nyc": "15.1.0",
"ts-loader": "8.1.0",
"ts-mocha": "8.0.0",
"typescript": "4.2.3",
"typescript": "4.2.4",
"webpack": "4.46.0"
}
}
2 changes: 1 addition & 1 deletion packages/opentelemetry-context-async-hooks/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@
"rimraf": "3.0.2",
"ts-mocha": "8.0.0",
"ts-node": "9.1.1",
"typescript": "4.2.3"
"typescript": "4.2.4"
},
"peerDependencies": {
"@opentelemetry/api": "^1.0.0-rc.0"
Expand Down
4 changes: 2 additions & 2 deletions packages/opentelemetry-context-zone-peer-dep/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
"access": "public"
},
"devDependencies": {
"@babel/core": "7.13.14",
"@babel/core": "7.13.15",
"@opentelemetry/api": "^1.0.0-rc.0",
"@types/mocha": "8.2.2",
"@types/node": "14.14.37",
Expand All @@ -65,7 +65,7 @@
"ts-loader": "8.1.0",
"ts-mocha": "8.0.0",
"ts-node": "9.1.1",
"typescript": "4.2.3",
"typescript": "4.2.4",
"webpack": "4.46.0",
"webpack-cli": "4.6.0",
"zone.js": "0.11.4"
Expand Down
6 changes: 3 additions & 3 deletions packages/opentelemetry-context-zone/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
"access": "public"
},
"devDependencies": {
"@babel/core": "7.13.14",
"@babel/core": "7.13.15",
"@types/mocha": "8.2.2",
"@types/node": "14.14.37",
"@types/sinon": "9.0.11",
Expand All @@ -58,13 +58,13 @@
"ts-loader": "8.1.0",
"ts-mocha": "8.0.0",
"ts-node": "9.1.1",
"typescript": "4.2.3",
"typescript": "4.2.4",
"webpack": "4.46.0",
"webpack-cli": "4.6.0",
"webpack-merge": "5.7.3"
},
"dependencies": {
"@opentelemetry/context-zone-peer-dep": "^0.18.2",
"@opentelemetry/context-zone-peer-dep": "0.18.2",
"zone.js": "^0.11.0"
},
"sideEffects": true
Expand Down
4 changes: 2 additions & 2 deletions packages/opentelemetry-core/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@
},
"devDependencies": {
"@opentelemetry/api": "^1.0.0-rc.0",
"@opentelemetry/propagator-b3": "^0.18.2",
"@opentelemetry/propagator-b3": "0.18.2",
"@types/mocha": "8.2.2",
"@types/node": "14.14.37",
"@types/semver": "7.3.4",
Expand All @@ -73,7 +73,7 @@
"ts-loader": "8.1.0",
"ts-mocha": "8.0.0",
"ts-node": "9.1.1",
"typescript": "4.2.3",
"typescript": "4.2.4",
"webpack": "4.46.0"
},
"peerDependencies": {
Expand Down
6 changes: 3 additions & 3 deletions packages/opentelemetry-core/src/utils/environment.ts
Original file line number Diff line number Diff line change
Expand Up @@ -101,9 +101,9 @@ export const DEFAULT_ENVIRONMENT: Required<ENVIRONMENT> = {
OTEL_PROPAGATORS: ['tracecontext', 'baggage'],
OTEL_RESOURCE_ATTRIBUTES: '',
OTEL_SAMPLING_PROBABILITY: 1,
OTEL_SPAN_ATTRIBUTE_COUNT_LIMIT: 1000,
OTEL_SPAN_EVENT_COUNT_LIMIT: 1000,
OTEL_SPAN_LINK_COUNT_LIMIT: 1000,
OTEL_SPAN_ATTRIBUTE_COUNT_LIMIT: 128,
OTEL_SPAN_EVENT_COUNT_LIMIT: 128,
OTEL_SPAN_LINK_COUNT_LIMIT: 128,
};

/**
Expand Down
20 changes: 10 additions & 10 deletions packages/opentelemetry-exporter-collector-grpc/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -45,9 +45,9 @@
"access": "public"
},
"devDependencies": {
"@babel/core": "7.13.14",
"@babel/core": "7.13.15",
"@opentelemetry/api": "^1.0.0-rc.0",
"@opentelemetry/api-metrics": "^0.18.2",
"@opentelemetry/api-metrics": "0.18.2",
"@types/mocha": "8.2.2",
"@types/node": "14.14.37",
"@types/sinon": "9.0.11",
Expand All @@ -61,18 +61,18 @@
"ts-loader": "8.1.0",
"ts-mocha": "8.0.0",
"ts-node": "9.1.1",
"typescript": "4.2.3"
"typescript": "4.2.4"
},
"peerDependencies": {
"@opentelemetry/api": "^1.0.0-rc.0"
},
"dependencies": {
"@grpc/proto-loader": "^0.5.4",
"@opentelemetry/core": "^0.18.2",
"@opentelemetry/exporter-collector": "^0.18.2",
"@opentelemetry/metrics": "^0.18.2",
"@opentelemetry/resources": "^0.18.2",
"@opentelemetry/tracing": "^0.18.2",
"grpc": "^1.24.2"
"@grpc/grpc-js": "^1.2.12",
"@grpc/proto-loader": "^0.6.0",
"@opentelemetry/core": "0.18.2",
"@opentelemetry/exporter-collector": "0.18.2",
"@opentelemetry/metrics": "0.18.2",
"@opentelemetry/resources": "0.18.2",
"@opentelemetry/tracing": "0.18.2"
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ import {
CollectorExporterBase,
collectorTypes,
} from '@opentelemetry/exporter-collector';
import type { Metadata } from 'grpc';
import type { Metadata } from '@grpc/grpc-js';
import {
CollectorExporterConfigNode,
GRPCQueueItem,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
*/

import { collectorTypes } from '@opentelemetry/exporter-collector';
import * as grpc from 'grpc';
import * as grpc from '@grpc/grpc-js';

/**
* Queue item to be used to save temporary spans/metrics in case the GRPC service
Expand Down
4 changes: 2 additions & 2 deletions packages/opentelemetry-exporter-collector-grpc/src/util.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
import { diag } from '@opentelemetry/api';
import * as protoLoader from '@grpc/proto-loader';
import { collectorTypes } from '@opentelemetry/exporter-collector';
import * as grpc from 'grpc';
import * as grpc from '@grpc/grpc-js';
import * as path from 'path';

import {
Expand Down Expand Up @@ -82,7 +82,7 @@ export function send<ExportItem, ServiceRequest>(

collector.serviceClient.export(
serviceRequest,
collector.metadata,
collector.metadata || new grpc.Metadata(),
(err: collectorTypes.ExportServiceError) => {
if (err) {
diag.error('Service request', serviceRequest);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ import { collectorTypes } from '@opentelemetry/exporter-collector';
import * as metrics from '@opentelemetry/metrics';
import * as assert from 'assert';
import * as fs from 'fs';
import * as grpc from 'grpc';
import * as grpc from '@grpc/grpc-js';
import * as path from 'path';
import * as sinon from 'sinon';
import { CollectorMetricExporter } from '../src';
Expand Down Expand Up @@ -109,9 +109,10 @@ const testCollectorMetricExporter = (params: TestParams) =>
]
)
: grpc.ServerCredentials.createInsecure();
server.bind(address, credentials);
server.start();
done();
server.bindAsync(address, credentials, () => {
server.start();
done();
});
});
});

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ import {

import * as assert from 'assert';
import * as fs from 'fs';
import * as grpc from 'grpc';
import * as grpc from '@grpc/grpc-js';
import * as path from 'path';
import * as sinon from 'sinon';
import { CollectorTraceExporter } from '../src';
Expand Down Expand Up @@ -104,9 +104,10 @@ const testCollectorExporter = (params: TestParams) =>
]
)
: grpc.ServerCredentials.createInsecure();
server.bind(address, credentials);
server.start();
done();
server.bindAsync(address, credentials, () => {
server.start();
done();
});
});
});

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ import * as metrics from '@opentelemetry/metrics';
import { Resource } from '@opentelemetry/resources';
import { ReadableSpan } from '@opentelemetry/tracing';
import * as assert from 'assert';
import * as grpc from 'grpc';
import * as grpc from '@grpc/grpc-js';

const meterProvider = new metrics.MeterProvider({
interval: 30000,
Expand Down
18 changes: 9 additions & 9 deletions packages/opentelemetry-exporter-collector-proto/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -45,9 +45,9 @@
"access": "public"
},
"devDependencies": {
"@babel/core": "7.13.14",
"@babel/core": "7.13.15",
"@opentelemetry/api": "^1.0.0-rc.0",
"@opentelemetry/api-metrics": "^0.18.2",
"@opentelemetry/api-metrics": "0.18.2",
"@types/mocha": "8.2.2",
"@types/node": "14.14.37",
"@types/sinon": "9.0.11",
Expand All @@ -61,18 +61,18 @@
"ts-loader": "8.1.0",
"ts-mocha": "8.0.0",
"ts-node": "9.1.1",
"typescript": "4.2.3"
"typescript": "4.2.4"
},
"peerDependencies": {
"@opentelemetry/api": "^1.0.0-rc.0"
},
"dependencies": {
"@grpc/proto-loader": "^0.5.4",
"@opentelemetry/core": "^0.18.2",
"@opentelemetry/exporter-collector": "^0.18.2",
"@opentelemetry/metrics": "^0.18.2",
"@opentelemetry/resources": "^0.18.2",
"@opentelemetry/tracing": "^0.18.2",
"@grpc/proto-loader": "^0.6.0",
"@opentelemetry/core": "0.18.2",
"@opentelemetry/exporter-collector": "0.18.2",
"@opentelemetry/metrics": "0.18.2",
"@opentelemetry/resources": "0.18.2",
"@opentelemetry/tracing": "0.18.2",
"protobufjs": "^6.9.0"
}
}
Loading

0 comments on commit be2e2b8

Please sign in to comment.