[Code Update - Chapter 09] Code update for MP Telemetry 2.1 - #72
Open
ttelang wants to merge 19 commits into
Open
[Code Update - Chapter 09] Code update for MP Telemetry 2.1#72ttelang wants to merge 19 commits into
ttelang wants to merge 19 commits into
Conversation
Updated sections on telemetry data exporting, metrics, and logs. Added new content for MicroProfile Telemetry 2.1 features.
- Clarified OpenTelemetry and MicroProfile Telemetry Integration. - Fixing grammatical & styling issues to ensure adherence to Eclipse Foundation Writing Style Guide throughout this chapter.
Update README.adoc as per the mp telemetry 2.1 implementation
ttelang
marked this pull request as ready for review
April 24, 2026 17:34
ttelang
changed the title
Emily-Jiang
requested changes
Apr 27, 2026
- Add docker-compose.yml with full LGTM stack (Grafana, Loki, Tempo, Prometheus, OTel Collector) - Add otel-collector-config.yml with correct OTLP exporter syntax and Prometheus port (8889) - Add prometheus.yml configured to scrape OTel Collector metrics endpoint - Enable logs exporter (otlp) and add Codespaces port-forward note in microprofile-config.properties Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ethods - Switch Tracer from GlobalOpenTelemetry.getTracer() to CDI @Inject per MP Telemetry 2.1 spec - Add CDI-injected Meter with paymentAttemptsCounter tracking success/failed/fallback outcomes - Add missing checkGatewayHealth() with @CIRCUITBREAKER called by PaymentResource - Add missing sendPaymentNotification() with @asynchronous @bulkhead called by PaymentResource - Remove type pom from microprofile-telemetry-api dependency, it is a JAR not a BOM - Enable otel.logs.exporter=otlp in README, was incorrectly set to none in two places
- Replace all Jaeger and Zipkin references in README with LGTM stack - Add Grafana/Tempo trace verification steps (port 13000) - Add Loki log correlation steps with trace-to-log linking - Add Prometheus metrics verification at port 19090 - Replace cleanup step (docker rm jaeger) with docker-compose down - Remove microprofile-telemetry-api JAR dependency from pom.xml; the artifact is only published as a POM on Maven Central, not a JAR. opentelemetry-api:1.48.0:provided covers all OTel types at compile time.
- Added `dns_search: []` to all services in `docker-compose.yml` to prevent DNS resolution issues in GitHub Codespaces. - Updated `otel-collector-config.yml` to use new OTLP endpoints for Tempo and Loki. - Introduced `tempo-config.yaml` for Grafana Tempo v3 compatibility. - Updated `README.adoc` to reflect changes in architecture and service descriptions. - Added images for Grafana dashboards related to payment service telemetry. - Changed OTLP endpoint in `microprofile-config.properties` to accommodate GitHub Codespaces.
Mp telemetry 2.1
…or fixing the github actions failure
Contributor
Author
|
@jclingan & @Emily-Jiang - I have resolved all the PR feedback; kindly approve and merge. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Below is the list of changes:
pom.xmlREADME.adocserver.xmlPaymentResource.javacheckGatewayHealth()and asynchronoussendNotification()method.micrprofile-config.propertiesindex.htmlFor your easy reference, here is the demo of how to run this code: https://youtube.com/live/K9qaq6BbaYM