otelhttp: make DefaultClient a non-mutable variable

[dmathieu]

DefaultClient is a mutable global variable - replace with something immutable or remove entirely

[KonstantinosTsitoumis]

Hello, may I pick this up?

[KonstantinosTsitoumis]

Here is the PR, Lmk what you think:)
#8140

[pellared]

I propose deprecating and later removing

[MrAlias]

I propose deprecating and later removing

Does this mean we should also remove the Get, Post, PostForm, and Head functions as well?

[KonstantinosTsitoumis]

Hello

• Could you please clarify a few things so I can plan the next steps properly?
• What should happen after the deprecation phase?
• How long should the deprecation period last before removing the old implementation?
• Is there a preferred way to communicate the deprecation?

This is my first ever contribution, so apologies if I’m a bit slow.

Thanks!

[dmathieu]

Since this is an unstable package, we only need to deprecate for one version before we clean up and remove the deprecated code.
Communicating through the changelog should be enough.

[KonstantinosTsitoumis]

The first step of the deprecation process is implemented in the PR.

[KonstantinosTsitoumis]

For the next change of removing the default client, did we have a specific reason that we wanted to have one instance of the client?? Should we create a new instance of client each time we want to do a request in each Post/Get etc func?