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

Merge the two versions of instrument.sh #3548

Open
Mpdreamz opened this issue Jul 29, 2024 · 4 comments
Open

Merge the two versions of instrument.sh #3548

Mpdreamz opened this issue Jul 29, 2024 · 4 comments
Milestone

Comments

@Mpdreamz
Copy link
Contributor

Currently we distribute two versions of instrument.sh

I believe we could reuse the redistributable instrument.sh with a few minor modifications:

  • Fallback to $BASE_PATH for OTEL_DOTNET_AUTO_HOME before falling back to $HOME.
  • Change test of OTEL_DOTNET_AUTO_HOME correctness to the presence of OpenTelemetry.AutoInstrumentation.Native.*. (possibly embedded in a $RUNTIME folder).

Happy to submit a PR. Opening an issue early to discuss if this is a good/bad idea 😸

@pjanotti
Copy link
Contributor

Hi @Mpdreamz - I like the idea, IIRC, it was something that came up during some PR review or SIG meeting. At first I don't see any reason to keep them separate.

@pjanotti
Copy link
Contributor

@Mpdreamz are you planning to start working on this? We may want to add it to our next milestone.

@Mpdreamz
Copy link
Contributor Author

@pjanotti I am :) Just opened #3553

I only validated the script still works when using a zip distribution thus far.

@pjanotti
Copy link
Contributor

Nice! Thanks @Mpdreamz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants