opentelemetry-erlang-contrib icon indicating copy to clipboard operation
opentelemetry-erlang-contrib copied to clipboard

Add otel tesla library

Open ricardoccpaiva opened this issue 3 years ago • 4 comments

OpenTelemetry Tesla

Tesla middleware that creates OpenTelemetry spans and injects tracing headers into HTTP requests for Tesla clients. Currently this library is located at https://github.com/ricardoccpaiva/opentelemetry_tesla

Checklist

Acknowledgements

  • [x] I certify that I am authorized to assign ownership of this code to the OpenTelemetry project.
  • [x] I acknowledge that ownership of the Hex package must be given/transferred to the OpenTelemetry Hex Org

Project Type

  • [x] Instrumentation
  • [ ] Exporter
  • [ ] Propagator
  • [ ] Metapackage
  • [ ] Example
  • [ ] Util

Prior Art

  • [ ] Is this an existing project? Includes private libraries being open sourced.
    • [x] Yes
    • [ ] No
  • [x] Are there any other packages on Hex providing similar functionality?

Existing Project

  • [x] Moving from an existing public repository? Provide a link in the description.
  • [x] All authors who have contributed code contained in this PR have signed the CNCF CLA

All Projects

  • [ ] Label created (maintainers)
  • [x] labeler.yml updated
  • [ ] CI
    • [x] Elixir or Erlang language file is updated
    • [ ] Test strategy should use the matrix strategy with all supported language/OTP combinations
    • [ ] The job is scoped to only run for relevant code changes
  • [x] Codeowners adds scoped ownership for project maintainers
  • [x] License file (Apache2 - copyright to OpenTelemetry Authors)
  • [x] All code complies with the Otel Specification and Otel Semantic Conventions - Trace Metrics

ricardoccpaiva avatar Jun 03 '22 22:06 ricardoccpaiva

CLA Signed

The committers listed above are authorized under a signed CLA.

  • :white_check_mark: login: ricardoccpaiva / name: Ricardo Paiva (cf4ad60847320a6f26c789f8d29f42f22f80cc21, 41d84b9cc44702d084577efeb60d99d0595a36ce, bebe5840ec7beca7885c8c470019354dc467ec71, 8560b1706ed3799ea9185918b9fcb3ae2bdc1525)

when should the hex.pm org be transferred? after merging?

ricardoccpaiva avatar Jun 03 '22 22:06 ricardoccpaiva

@tsloughter @bryannaegele can I get a review on this? 🙏

ricardoccpaiva avatar Jul 07 '22 14:07 ricardoccpaiva

@ricardoccpaiva sorry about that! I think this is good to go. I just had one question.

tsloughter avatar Aug 02 '22 23:08 tsloughter

@ricardoccpaiva now that this is merged, can you please transfer package ownership to the opentelemetry? Thank you!

bryannaegele avatar Aug 26 '22 19:08 bryannaegele

@ricardoccpaiva now that this is merged, can you please transfer package ownership to the opentelemetry? Thank you!

yes! it should be using the mix hex.owner command, right? is there anything else needed?

ricardoccpaiva avatar Aug 30 '22 09:08 ricardoccpaiva

That's it! It would be mix hex.owner transfer opentelemetry_tesla opentelemetry

bryannaegele avatar Aug 30 '22 14:08 bryannaegele

That's it! It would be mix hex.owner transfer opentelemetry_tesla opentelemetry

done ✅

ricardoccpaiva avatar Aug 31 '22 11:08 ricardoccpaiva