-
Notifications
You must be signed in to change notification settings - Fork 45
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 WIP instrumentation quickstart sample #326
Add WIP instrumentation quickstart sample #326
Conversation
The docs under docs/examples are for https://google-cloud-opentelemetry.readthedocs.io/en/latest/. Let just leave it in /samples for now |
I'm going to switch to manual instrumentation |
84ccaa1
to
a2227b0
Compare
I think this is ready for review. The app builds and runs successfully, but it does not include any instrumentation. I'll do that as follow-ups |
@aabmass any recommendation on testing this as part of CI? Should I use cloud build similar to the java quickstart? |
Co-authored-by: Aaron Abbott <[email protected]>
This is mostly boilerplate to make future PRs smaller.
The following are taken from #309:
samples/instrumentation-quickstart/Dockerfile
samples/instrumentation-quickstart/app.py
The following are minimally adapted from the java quickstart:
The quickstart runs and serves requests, but does not currently include any instrumentation. The logs are also not in the correct format.
I noticed we have some examples in docs/examples, and another example in /samples. Should we consolidate?