DB-API Instrumentor `trace_integration` doesn't support all `wrap_connect` args.

[tammy-baylis-swi]

What problem do you want to solve?

DB-API Instrumentor's wrap_connect is a common util used by concrete db client framework instrumentors (e.g. psycopg2, mysqlclient) to wrap database connections to instrument queries.

trace_integration is a thin wrapper around wrap_connect. It has been documented for a long time so it's likely being used manually in the wild, just not by instrumentors e.g. auto-instrumentation. trace_integration accepts all the same args as wrap_connect except commenter_options.

Describe the solution you'd like

Update trace_integration to accept option commenter_options for consistent behaviour. The documentation (readthedocs) should also be updated.

Describe alternatives you've considered

No response

Additional Context

Noticed during documentation update in #3720

Would you like to implement a fix?

None

Tip

_{React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it. Learn more here.}

[chloe-zh97]

Hi @tammy-baylis-swi , I am new to open source contribution and willing to help, can I pick up the issue?

[tammy-baylis-swi]

Hi @chloe-zh97 , welcome! Yes, I have assigned this issue to you.

[chloe-zh97]

Hi @tammy-baylis-swi , I am ready to create the PR, but got 403, if I don't miss any pre step, can you help with grant the access? Thanks!

[tammy-baylis-swi]

Hi @chloe-zh97 , hmm a 403 isn't good. Could you try the following if you haven't already?:

1. Read and sign the CLA so it's associated with your github account and commits: https://easycla.lfx.linuxfoundation.org/#/?version=2
2. You'll have to fork the repo to your github account. Then your PR will be from a branch on your fork into the main branch of the contrib repo. Here are more detailed steps: https://github.com/open-telemetry/opentelemetry-python-contrib/blob/main/CONTRIBUTING.md#pull-requests.