Commit 8e68031
committed
fix: Move the logs-sdk and metrics-sdk requires
This makes it so the logs and metrics will work when you require like
```ruby
require 'opentelemetry/sdk/logs'
require 'opentelemetry/sdk/metrics'
```
Without getting 'opentelemetry-logs-api' or 'opentelemetry-logs-metrics'
earlier.
The opentelemetry-exporter-otlp-logs gem works around this, but I'm
leaving that workaround there for now, so it will work with old versions
of the opentelemetry-logs-sdk gem that are still locked.
Also added a missing `require 'opentelemetry/sdk/metrics/version'`.
[Fixes #1955]1 parent 19ad25d commit 8e68031
File tree
4 files changed
+7
-4
lines changed- logs_sdk/lib
- opentelemetry/sdk
- metrics_sdk/lib
- opentelemetry/sdk
4 files changed
+7
-4
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
8 | | - | |
9 | 7 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
7 | 10 | | |
8 | 11 | | |
9 | 12 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
8 | | - | |
9 | 7 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
7 | 10 | | |
8 | 11 | | |
9 | 12 | | |
| |||
13 | 16 | | |
14 | 17 | | |
15 | 18 | | |
| 19 | + | |
16 | 20 | | |
17 | 21 | | |
18 | 22 | | |
| |||
0 commit comments