-
Notifications
You must be signed in to change notification settings - Fork 791
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
Add documentation for spring starters #1035
Add documentation for spring starters #1035
Conversation
9afdf39
to
dc2b77a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thx @mabdinur! great docs!
instrumentation-core/spring/spring-boot-autoconfigure/README.md
Outdated
Show resolved
Hide resolved
|
||
### Add these dependencies to your project. | ||
|
||
Replace `OPENTELEMETRY_VERSION` with the latest stable [release](https://mvnrepository.com/artifact/io.opentelemetry). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think better to link to official site, e.g. https://search.maven.org/search?q=g:io.opentelemetry
also, we aren't publishing to maven central yet from opentelemetry-java-instrumentation, so maybe include somewhere that for now you need to add our bintray maven repository: https://dl.bintray.com/open-telemetry/maven/
<version>OPENTELEMETRY_VERSION</version> | ||
</dependency> | ||
|
||
<!-- Slf4j log correlation support --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we have this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not yet. This work is blocked by this issue: open-telemetry/opentelemetry-java#922. I left it in as a placeholder. But since logging support is not required for GA I don't think this issue will be resolve in the next few weeks. I'll remove this for now.
instrumentation-core/spring/spring-boot-autoconfigure/README.md
Outdated
Show resolved
Hide resolved
instrumentation-core/spring/spring-boot-autoconfigure/README.md
Outdated
Show resolved
Hide resolved
instrumentation-core/spring/starters/jaeger-exporter-starter/README.md
Outdated
Show resolved
Hide resolved
instrumentation-core/spring/starters/otlp-exporter-starter/README.md
Outdated
Show resolved
Hide resolved
instrumentation-core/spring/starters/zipkin-exporter-starter/README.md
Outdated
Show resolved
Hide resolved
|Feature |Property |Default Value | ||
|--- |--- |--- | ||
|Otlp Exporter |opentelemetry.trace.exporters.otlp.servicename |OtlpGrpcSpanExporter.DEFAULT_SERVICE_NAME | ||
| |opentelemetry.trace.exporters.otlp.endpoint |OtlpGrpcSpanExporter.DEFAULT_ENDPOINT | ||
| |opentelemetry.trace.exporters.otlp.spantimeout |OtlpGrpcSpanExporter.DEFAULT_DEADLINE_MS | ||
|Jaeger Exporter |opentelemetry.trace.exporters.jaeger.servicename |JaegerGrpcSpanExporter.DEFAULT_SERVICE_NAME | ||
| |opentelemetry.trace.exporters.jaeger.endpoint |JaegerGrpcSpanExporter.DEFAULT_ENDPOINT | ||
| |opentelemetry.trace.exporters.jaeger.spantimeout |JaegerGrpcSpanExporter.DEFAULT_DEADLINE_MS | ||
|Zipkin Exporter |opentelemetry.trace.exporters.jaeger.servicename |ZipkinSpanExporter.DEFAULT_SERVICE_NAME | ||
| |opentelemetry.trace.exporters.jaeger.endpoint |ZipkinSpanExporter.DEFAULT_ENDPOINT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for the documentation, i think nice to include the real default values
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
…EADME.md Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
…DME.md Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
…EADME.md Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sweet
instrumentation-core/spring/spring-boot-autoconfigure/README.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Anuraag Agrawal <anuraaga@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @mabdinur!
Add documentation for:
Opentelemetry-spring-boot-autoconfigure
Opentelemetry-spring-starter
Opentelemetry-zipkin-exporter-starter
Opentelemetry-jaeger-exporter-starter
Opentelemetry-otlp-exporter-starter
Minor clean up: I removed 2 fields from WithSpanAspectTest that are not being used in any tests.