You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2022/08/04 07:22:53 UTC
[GitHub] [camel-quarkus-examples] llowinge opened a new pull request, #100: Fix example for ability to use Jaeger exporter only
llowinge opened a new pull request, #100:
URL: https://github.com/apache/camel-quarkus-examples/pull/100
Note that the `main` branch points at the latest stable Camel Quarkus release.
Pull requests should be generally send against the `camel-quarkus-main` branch pointing at the current Camel Quarkus SNAPSHOT.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus-examples] jamesnetherton commented on a diff in pull request #100: Fix example for ability to use Jaeger exporter only
Posted by GitBox <gi...@apache.org>.
jamesnetherton commented on code in PR #100:
URL: https://github.com/apache/camel-quarkus-examples/pull/100#discussion_r937719956
##########
observability/README.adoc:
##########
@@ -63,6 +63,26 @@ You can also directly leverage MicroProfile Health APIs to create checks. Class
The tracing configuration for the application can be found within `application.properties`.
+Default configuration uses OTLP exporter, but it can be easily switched to Jaeger exporter with applying this change in `application.properties`:
Review Comment:
```suggestion
The default configuration uses the OTLP exporter, but it can be easily switched to the Jaeger exporter by applying this change in `application.properties`:
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus-examples] llowinge commented on a diff in pull request #100: Fix example for ability to use Jaeger exporter only
Posted by GitBox <gi...@apache.org>.
llowinge commented on code in PR #100:
URL: https://github.com/apache/camel-quarkus-examples/pull/100#discussion_r937728486
##########
observability/README.adoc:
##########
@@ -63,6 +63,26 @@ You can also directly leverage MicroProfile Health APIs to create checks. Class
The tracing configuration for the application can be found within `application.properties`.
+Default configuration uses OTLP exporter, but it can be easily switched to Jaeger exporter with applying this change in `application.properties`:
Review Comment:
Fixed, thanks @jamesnetherton
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus-examples] jamesnetherton merged pull request #100: Fix example for ability to use Jaeger exporter only
Posted by GitBox <gi...@apache.org>.
jamesnetherton merged PR #100:
URL: https://github.com/apache/camel-quarkus-examples/pull/100
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org