opentelemetry-sqlcommenter

opentelemetry-sqlcommenter copied to clipboard

- Migrating index doc & samples from google-cloud to opentelemetry

Thiyagu55

Plan for ci/infra/release

9

I am starting this discussion to consolidate our thoughts/ideas and prepare a plan ahead. I have a couple of questions in my mind that I think should be brought up....

srikanthccv

[Javascript] Consider underlying db package instead of ORM

3

I have been thinking about `sqlcommenter`. I was wondering whether it would make more sense to instrument the underlying packages used to talk to the databases instead of instrumenting the...

weyert

Add proper self contained examples/tutorials for all languages and ORMs

1

Would it be a good idea to have self contained examples and tutorials for a new user to quickly try out sqlcommenter. My thought process is something like a docker-compose/kubernetes...

sjs994

Setup GitHub Actions CI

1

TO-DO - [ ] py - [ ] rb - [ ] js - [ ] java

srikanthccv

At present, we are using Java8. Need to upgrade it to Java 17 to match with opentelemetry-java.

sjs994

Add FastAPI implementation to OTEL

1

Fixes https://github.com/open-telemetry/opentelemetry-sqlcommenter/issues/40 This copies over the existing fastapi implementation from https://github.com/google/sqlcommenter/tree/master into the opentelemetry-sqlcommenter library since I believe this was originally a 1:1 copy and just hasn't been kept up...

ThomasWunderlich

sqlcommenter has support for fastapi in https://github.com/google/sqlcommenter/blob/master/python/sqlcommenter-python/google/cloud/sqlcommenter/fastapi.py This should be ported over to opentelemetry so OTEL users have the same functionality

ThomasWunderlich

See https://github.com/open-telemetry/community/issues/2636 for details

opentelemetrybot

See https://github.com/open-telemetry/community/issues/2574 for details

opentelemetrybot