* First version of WithSpan annotation.
This should be followed by the corresponding PR in auto-instrumentation to actually create spans around annotated methods.
* Add license
* Code style fix
* Rename module, add documentation.
* License added.
* Update contrib/auto_annotations/src/main/java/io/opentelemetry/contrib/auto/annotations/WithSpan.java
Co-Authored-By: John Watson <jkwatson@gmail.com>
* Apply suggestions from code review
Co-Authored-By: John Watson <jkwatson@gmail.com>
* Code style fix
Co-authored-by: John Watson <jkwatson@gmail.com>
* Add error tag to spans for Jaeger exporter
The auto-instrumentation libraries add errors to the spans when they occur. The error tag is not added to the spans by the Jaeger exporter if the span status is not OK.
When sending these spans to the OpenTelemetry Collector it will add the error tag when it translates them and sends them to the Jaeger collector. This process does not
take place when sending spans directly to a Jaeger collector (not using the OTel collector) and so the Jaeger UI does not display visual indicators for spans with errors.
Adding the error tag in the adapter if the span.status.code is not OK, and this tag hasn't already been set, means that it will be present when using either the OTel collector
or the Jaeger collector directly. The Otel collector will remove this tag automatically when it receives the span and sets the status code so this will not be sent as a duplicate.
* Remove additional attribute check in span status condition
Co-Authored-By: Trask Stalnaker <trask.stalnaker@gmail.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
* Add Jaeger remote sampler
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
* Add tests
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
* Add remote sampler to all BOM
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
* Make docker test optional
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
* Use scheduled thread pool
* Use daemon thread factory
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
* Cosmetic changes
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
* Add readme
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
* add the sampling probability to the decision, when appropriate.
* formatting
* normalize a method name and move attribute to semantic convention attributes
* move the sampling attribute to its own class in the SDK
* Move the sampling priority attribute to a non-public spot
* consolidated benchmarks for contributed propagators, included b3 propagator in benchmarks
* added comments, also added extra benchmark to test injecting into b3 multiple headers
* Mark all threads/pools as daemon.
* Move setDaemon calls inside try blocks as they can throw a security exception.
* Add javadoc comment about possibility of SecurityException.
* Create common/DaemonThreadFactory and adjust all thread creation to use it.
* Apply a name prefix in the DaemonThreadPool; rework and simplify several calling sites accordingly.
* Provides a way to disable the docker integration tests via a gradle property.
* switch the property to be the opposite
* add a new make target and let CI use it.
* Update gradle.properties
Co-Authored-By: Armin Ruech <armin.ruech@gmail.com>
* Update exporters/jaeger/src/test/java/io/opentelemetry/exporters/jaeger/JaegerIntegrationTest.java
Co-Authored-By: Armin Ruech <armin.ruech@gmail.com>
* Update exporters/jaeger/src/test/java/io/opentelemetry/exporters/jaeger/JaegerIntegrationTest.java
Co-Authored-By: Armin Ruech <armin.ruech@gmail.com>
* simplify the test flow
Co-authored-by: Armin Ruech <armin.ruech@gmail.com>
The OpenTelemetry Span.Kind enumerated type values are in uppercase, e.g. SERVER, CLIENT. These are incompatible with Jaeger which is based on the OpenTracing semantics where these are in lowercase.
The spans are collected as expected, but Jaeger is unable to display deep dependancy graphs as these are based on span kind values in lowercase, e.g. "server".
* added jaeger propogator and test
* address PR comments about logging and static var names
* added jaeger client for testing jaeger propagator
* added comment to explain why UTF-8 decoding is needed
* Promote John to maintainer
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
* Update README.md
Co-Authored-By: John Watson <jkwatson@gmail.com>
Co-authored-by: John Watson <jkwatson@gmail.com>
* Initial def of string constants for semantic convention span attribute names
* Change to recommended name and update to latest specifications
* Changed constants to objects which enabled typed setting of span attributes
* Sync Javadoc with specifications document.
* Update to latest factory name
* Renaming to make usage clearer
* API improvements and cleanup based on PR comments
* Removed methods required by code reviews
* Made setter classes final
* #1002 Fill in the Metrics details in the Getting Started guide.
Signed-off-by: Sergei Malafeev <sergeymalafeev@gmail.com>
* #1002 implement suggested changes
Signed-off-by: Sergei Malafeev <sergeymalafeev@gmail.com>
* #1002 remove set label key
Signed-off-by: Sergei Malafeev <sergeymalafeev@gmail.com>
* #1002 add clarification to counter bind
Signed-off-by: Sergei Malafeev <sergeymalafeev@gmail.com>
Trask Stalnaker