-
Notifications
You must be signed in to change notification settings - Fork 50
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
OpenTelemetry sample #18
Conversation
1ad7c8f
to
adcdd86
Compare
# Conflicts: # README.md # poetry.lock
metrics: | ||
receivers: [otlp] | ||
exporters: [logging] | ||
processors: [batch] |
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.
newline?
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 following exactly, but definitely no need for a newline at the end of the YAML file (also no harm either of course)
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.
yeah not need, just some things complain like github complain if there is no newline at the end of a file
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 it's more of a "notice" than a complaint (same with git
diff in general since humans can't otherwise tell)
To run, first see [README.md](../README.md) for prerequisites. Then run the following to start a Jaeger container to | ||
view the trace results: | ||
|
||
docker run -d --name jaeger \ |
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.
does a user need to install this container separately?
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.
No, the container is automatically downloaded if not present by docker
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.
Looks good to me, just a few small question
What was changed
Add OpenTelemetry sample
Checklist