Skip to content
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

feat(mockotlpserver): use diagnostic channels for passing parsed OTLP data to printers #8

Merged
merged 1 commit into from
Dec 21, 2023

Conversation

trentm
Copy link
Member

@trentm trentm commented Dec 21, 2023

The HTTP and gRPC servers will now publish parsed OTLP data to
diagnostics channels named 'otlp.v1.trace', etc.
Then there is a simple Printer class which supports a subclass
with 'printTrace' et al methods. Those will subscribe to the
'otlp.*' channels and print as needed.

Multiple independent subscribers can be added. We can change
the 'Printer' name later if something else make better sense.

Using diagnostics_channel required bumping the min node to 14.17.0.

The only current printer is the InspectPrinter.

… data to printers

The HTTP and gRPC servers will now publish parsed OTLP data to
diagnostics channels named 'otlp.v1.trace', etc.
Then there is a simple Printer class which supports a subclass
with 'printTrace' et al methods. Those will subscribe to the
'otlp.*' channels and print as needed.

Multiple independent subscribers can be added. We can change
the 'Printer' name later if something else make better sense.

Using diagnostics_channel required bumping the min node to 14.17.0.

The only current printer is the InspectPrinter.
@trentm trentm self-assigned this Dec 21, 2023
@david-luna david-luna merged commit 7486ff3 into main Dec 21, 2023
1 check passed
@david-luna david-luna deleted the trentm/mos-diagch branch December 21, 2023 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants