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

Provide easy way to install relevant instrumentation #606

Closed
codeboten opened this issue Apr 21, 2020 · 4 comments
Closed

Provide easy way to install relevant instrumentation #606

codeboten opened this issue Apr 21, 2020 · 4 comments
Assignees

Comments

@codeboten
Copy link
Contributor

It would be convenient for users to be able to install instrumenting libraries which are relevant to them. This would simplify the initial experience with OpenTelemetry. Added PR #605 request with one option to install all existing instrumentation. The ruby SIG solves this problem in a similar fashion to that PR open-telemetry/opentelemetry-ruby#223

@owais
Copy link
Contributor

owais commented Apr 23, 2020

SignalFx Python instrumentation library ships with a bootstrap command that detects installed libraries and installs instrumentation packages for them. The workflow look roughly like the following:

pip install signalfx-tracing
sfx-py-trace-bootstrap

I think a command like this would be very useful. SignalFx version automatically installs the packages but I'd probably make it spit out new requirements instead by default. It could have flags to let users choose whether they want it to auto-install packages, just print them to console or directly modify a requirements.txt file.

ref: https://github.com/signalfx/signalfx-python-tracing/tree/master/scripts#sfx-py-trace-bootstrap

@owais
Copy link
Contributor

owais commented May 5, 2020

PR: #650

@lzchen
Copy link
Contributor

lzchen commented Nov 9, 2020

@codeboten

Is this issue solved with #650 ?

@codeboten
Copy link
Contributor Author

Yup, closing

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 a pull request may close this issue.

3 participants