Skip to content
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

feat: Collector Metric Exporter for the Web #1308

Merged
merged 51 commits into from
Aug 5, 2020
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
51 commits
Select commit Hold shift + click to select a range
7482b43
Merged from upstream
davidwitten Jul 10, 2020
b02949a
Add test
davidwitten Jul 11, 2020
5bd4fef
Basic node
davidwitten Jul 12, 2020
a002c1d
Tests and functions
davidwitten Jul 13, 2020
1d28df4
minor
davidwitten Jul 13, 2020
cf100b3
new line
davidwitten Jul 13, 2020
6a3e948
DefaultURL
davidwitten Jul 13, 2020
4eb3831
Add JSON tests
davidwitten Jul 13, 2020
547e63b
Test rename
davidwitten Jul 13, 2020
b08de2c
Refactor a lot
davidwitten Jul 14, 2020
dffc3dd
Restored those files
davidwitten Jul 14, 2020
9c9abae
Variable anme
davidwitten Jul 14, 2020
589a0e1
Merge remote-tracking branch 'upstream/master' into browser
davidwitten Jul 15, 2020
6c98095
Duplicate test
davidwitten Jul 15, 2020
0ecb031
Fixed strings
davidwitten Jul 15, 2020
629fa50
Changed metrics to functions and added summary
davidwitten Jul 15, 2020
dc1a492
Fixed tests
davidwitten Jul 16, 2020
5e040b1
Merge remote-tracking branch 'upstream/master' into summary
davidwitten Jul 17, 2020
f08faf3
Merge remote-tracking branch 'upstream/master' into node
davidwitten Jul 17, 2020
12d978c
Merge remote-tracking branch 'upstream/master' into browser
davidwitten Jul 17, 2020
a0bb009
Conflicts
davidwitten Jul 17, 2020
40ef4d4
Conflicts
davidwitten Jul 17, 2020
37f154b
Added browser
davidwitten Jul 17, 2020
707427f
Merge remote-tracking branch 'upstream/master' into everything
davidwitten Jul 20, 2020
70e3061
Merge remote-tracking branch 'upstream/master' into browser
davidwitten Jul 20, 2020
87e3900
Hierarchy
davidwitten Jul 20, 2020
f89378b
Lint
davidwitten Jul 20, 2020
7b90b2c
Rename var
davidwitten Jul 20, 2020
07d0b94
Remove reset
davidwitten Jul 21, 2020
fb9b74c
Reset
davidwitten Jul 21, 2020
b6c7b77
Merge remote-tracking branch 'upstream/master' into browser
davidwitten Jul 24, 2020
495f1c6
Merged
davidwitten Jul 27, 2020
b2e40d0
update submodule
davidwitten Jul 27, 2020
13eda4a
Newline
davidwitten Jul 27, 2020
99abf64
Lint
davidwitten Jul 27, 2020
3ae0247
Documentatiom
davidwitten Jul 28, 2020
7ba0b66
Example
davidwitten Jul 29, 2020
9c292d7
fix: verified example
davidwitten Jul 29, 2020
e61e809
merged
davidwitten Jul 29, 2020
765ddfa
Newline
davidwitten Jul 29, 2020
10236ee
feat: simplified example
davidwitten Jul 29, 2020
44eab55
chore: merge
davidwitten Jul 29, 2020
21c9e8b
chore: rename start
davidwitten Jul 31, 2020
a8857d3
refactor: split up files
davidwitten Jul 31, 2020
15de18b
fix: lint
davidwitten Jul 31, 2020
175e050
fix: add comments
davidwitten Jul 31, 2020
e526e11
fix: private
davidwitten Aug 3, 2020
6d3a080
Merge branch 'master' into browser
dyladan Aug 4, 2020
d29be93
Merge remote-tracking branch 'upstream/master' into browser
davidwitten Aug 4, 2020
ccedacc
fix: readme
davidwitten Aug 4, 2020
775d2a4
Merged
davidwitten Aug 4, 2020
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions examples/collector-exporter-node/docker/collector-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,8 @@ receivers:
exporters:
zipkin:
endpoint: "http://zipkin-all-in-one:9411/api/v2/spans"
prometheus:
endpoint: "0.0.0.0:9464"

processors:
batch:
Expand All @@ -21,3 +23,7 @@ service:
receivers: [otlp]
exporters: [zipkin]
processors: [batch, queued_retry]
metrics:
receivers: [otlp]
exporters: [prometheus]
processors: [batch, queued_retry]
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ services:
volumes:
- ./collector-config.yaml:/conf/collector-config.yaml
ports:
- "9464:9464"
- "55680:55680"
- "55681:55681"
depends_on:
Expand Down
9 changes: 5 additions & 4 deletions examples/collector-exporter-node/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,11 @@
"url": "https://github.com/open-telemetry/opentelemetry-js/issues"
},
"dependencies": {
"@opentelemetry/api": "^0.9.0",
"@opentelemetry/core": "^0.9.0",
"@opentelemetry/exporter-collector": "^0.9.0",
"@opentelemetry/tracing": "^0.9.0"
"@opentelemetry/api": "^0.10.1",
"@opentelemetry/core": "^0.10.1",
"@opentelemetry/exporter-collector": "^0.10.1",
"@opentelemetry/metrics": "^0.10.1",
"@opentelemetry/tracing": "^0.10.1"
},
"homepage": "https://github.com/open-telemetry/opentelemetry-js#readme"
}
4 changes: 3 additions & 1 deletion examples/collector-exporter-node/start.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ const opentelemetry = require('@opentelemetry/api');
// const { ConsoleLogger, LogLevel} = require('@opentelemetry/core');
const { BasicTracerProvider, ConsoleSpanExporter, SimpleSpanProcessor } = require('@opentelemetry/tracing');
const { CollectorTraceExporter, CollectorMetricExporter, CollectorProtocolNode } = require('@opentelemetry/exporter-collector');
const { MeterProvider } = require('@opentelemetry/metrics');
davidwitten marked this conversation as resolved.
Show resolved Hide resolved

const exporter = new CollectorTraceExporter({
serviceName: 'basic-service',
Expand All @@ -17,6 +18,7 @@ const exporter = new CollectorTraceExporter({

const metricExporter = new CollectorMetricExporter({
serviceName: 'basic-metric-service',
// logger: new ConsoleLogger(LogLevel.DEBUG),
});

const provider = new BasicTracerProvider();
Expand Down Expand Up @@ -87,4 +89,4 @@ const labels = { pid: process.pid, environment: 'staging' };
setInterval(() => {
requestCounter.bind(labels).add(1);
upDownCounter.bind(labels).add(Math.random() > 0.5 ? 1 : -1);
}, 1000);
}, 1000);