nginx module about opentelemetry-cpp-contrib HOT 3 CLOSED

Thanks @seemk for initiating first contribution for this repo :)

My suggestion would be to have single mono-repo with below set of sub-directories to start with:

• instrumentation
• exporters
• propagators

And the under Instrumentation, we can have further:

• instrumentation/nginx
• instrumentation/apache
• instrumentation/grpc

I do understand it would be difficult to manage them down the line as the contributions increases, and that time we can decide on having CODEOWNERS for each of the contrib directories. This is how it is done in opentelemetry-collector-contrib: https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/.github/CODEOWNERS. Or we can even do this right away i.e, - the original contributor(s) for a module will be the CODEOWNERS for that.

The releases would be source distribution for entire repo. CI would build and test required set of contrib modules, and not everything.
Let's hear from others too.

@maxgolov There is a PR for initial directory structure raised by you. Would you like to address the comments so that we can merge the changes to get started.

from opentelemetry-cpp-contrib.

Meanwhile, I would suggest you to raise a PR as discussed above, and let others comment over there.

from opentelemetry-cpp-contrib.

Thanks @lalitb , I've opened a PR: #5

from opentelemetry-cpp-contrib.