opentelemetry-python icon indicating copy to clipboard operation
opentelemetry-python copied to clipboard

OTLP Exporters API documentation page is missing the API section

Open MoshiBin opened this issue 3 years ago • 3 comments

The API section in the OpenTelemetry OTLP Exporters documentation page is not being generated. According to otlp.rst, it should be generated:

.. automodule:: opentelemetry.exporter.otlp.proto.grpc
    :members:
    :undoc-members:
    :show-inheritance:

MoshiBin avatar Jun 28 '22 14:06 MoshiBin

I think it is behaving as has been defined, it just looks like this hasn't been updated as this functionality was probably split into a few submodules? You can see the contents of what is currently displayed via the UI here (https://github.com/open-telemetry/opentelemetry-python/blob/b9a6358583c0676482a043279c01b252dc6c308e/exporter/opentelemetry-exporter-otlp-proto-grpc/src/opentelemetry/exporter/otlp/proto/grpc/init.py), but there is no code content for it to build the API docs from, it looks as though this has been split into trace_exporter, metric_exporter and _log_exporter but without any documentation currently.

I have raised the above PR to show what I mean, but I am new to opentelemetry and only at a basic scripting level with python and associated tooling, as opposed to being a guru so going to have to figure out how to go about testing this to ensure it behaves as expected unless anyone has any recommendations / can point me in the right direction of course!

RobsonSutton avatar Jul 23 '22 17:07 RobsonSutton

If you run the tox command tox -e docs it should error if there is any issue with documentation. Otherwise, it will create a _build/html folder which contains the API documentation file. You should be able to browse the generated documentation by opening index.html and see if did what you expected it to do.

srikanthccv avatar Jul 23 '22 18:07 srikanthccv

@srikanthccv - Thank you for the comment! I'll take a look at this later today and hopefully tidy everything up a bit more and have it appropriately tested 👍

RobsonSutton avatar Jul 25 '22 08:07 RobsonSutton