Skip to content

Commit

Permalink
Clarify SpanMetrics Connector migration
Browse files Browse the repository at this point in the history
* Add explicit reference to adding a new CLI flag
  * Resolves jaegertracing/jaeger#4502
* Fix `[x][y]` links that are not rendered correctly in `{{< info >}}` banner

Signed-off-by: Yuri Shkuro <[email protected]>
  • Loading branch information
yurishkuro committed Jun 5, 2023
1 parent 48388e8 commit ef5dd20
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
4 changes: 3 additions & 1 deletion content/docs/1.46/spm.md
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,9 @@ can be started via docker. Be sure to include `--net monitor_backend` in the `do
## Architecture

{{< info >}}
Starting with v1.46.0, Jaeger supports the OpenTelemetry [SpanMetrics Connector][spanmetrics-conn], which is replacing the deprecated [SpanMetrics Processor][spanmetrics]. Please refer to the [migration guide](https://github.com/jaegertracing/jaeger/blob/main/docker-compose/monitor/README.md#migrating-to-span-metrics-connector).
Starting with v1.46.0, Jaeger supports the OpenTelemetry [SpanMetrics Connector](https://pkg.go.dev/github.com/open-telemetry/opentelemetry-collector-contrib/connector/spanmetricsconnector#section-readme), which is replacing the deprecated [SpanMetrics Processor](https://pkg.go.dev/github.com/open-telemetry/opentelemetry-collector-contrib/processor/). Please refer to the [migration guide](https://github.com/jaegertracing/jaeger/blob/main/docker-compose/monitor/README.md#migrating-to-span-metrics-connector).

NB: Use `--prometheus.query.support-spanmetrics-connector=true` to explicitly enable the SpanMetrics Connector. This will become the default behavior in the future.
{{< /info >}}

The RED metrics queried by Jaeger for the Monitor tab are the result of span
Expand Down
4 changes: 3 additions & 1 deletion content/docs/next-release/spm.md
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,9 @@ can be started via docker. Be sure to include `--net monitor_backend` in the `do
## Architecture

{{< info >}}
Starting with v1.46.0, Jaeger supports the OpenTelemetry [SpanMetrics Connector][spanmetrics-conn], which is replacing the deprecated [SpanMetrics Processor][spanmetrics]. Please refer to the [migration guide](https://github.com/jaegertracing/jaeger/blob/main/docker-compose/monitor/README.md#migrating-to-span-metrics-connector).
Starting with v1.46.0, Jaeger supports the OpenTelemetry [SpanMetrics Connector](https://pkg.go.dev/github.com/open-telemetry/opentelemetry-collector-contrib/connector/spanmetricsconnector#section-readme), which is replacing the deprecated [SpanMetrics Processor](https://pkg.go.dev/github.com/open-telemetry/opentelemetry-collector-contrib/processor/). Please refer to the [migration guide](https://github.com/jaegertracing/jaeger/blob/main/docker-compose/monitor/README.md#migrating-to-span-metrics-connector).

NB: Use `--prometheus.query.support-spanmetrics-connector=true` to explicitly enable the SpanMetrics Connector. This will become the default behavior in the future.
{{< /info >}}

The RED metrics queried by Jaeger for the Monitor tab are the result of span
Expand Down

0 comments on commit ef5dd20

Please sign in to comment.