sdk/__init__.py not correctly packaged

[ffe4]

Describe your environment
macOS-10.15.5
Python 3.8
opentelemetry-sdk==0.12b0

Steps to reproduce

$ pip install opentelemetry-sdk
$ python -c 'import site, os; path = site.getsitepackages()[0]; print(os.path.exists(path + "/opentelemetry/sdk/"))'
True
$ python -c 'import site, os; path = site.getsitepackages()[0]; print(os.path.exists(path + "/opentelemetry/sdk/__init__.py"))'
False

What is the expected behavior?
I expected the __init__.py from opentelemetry-sdk/src/opentelemetry/sdk/__init__.py in the repository to be present.

What is the actual behavior?
The file was not there.

[ocelotl]

Sorry, I remembered something about a way to include files in the package but it was for additional files, as explained here.

[ffe4]

For context:

I encountered this issue while testing PR #1069. The tests pass when running python eachdist.py test, but fail in the CI, as well as when running tox -e py38-test-core-sdk locally.

Tox installs the package without the -e flag to test the actual installation.

[aabmass]

Looks like this line is the issue:

opentelemetry-python/opentelemetry-sdk/setup.cfg

Line 49 in 65ddc2d

include = opentelemetry.sdk.*

The reason it fails passes in tox is because tox.ini is using not an editable install (which doesn't uses the setuptools configuration afaik):

opentelemetry-python/tox.ini

Line 241 in 65ddc2d

test: pip install {toxinidir}/opentelemetry-api {toxinidir}/opentelemetry-sdk {toxinidir}/tests/util

[aabmass]

Strangely, it also looks like the version.py files are missing in a fresh install.

python3 -m venv venv
source venv/bin/activate
pip install opentelemetry-api opentelemetry-sdk
tree -I *.pyc -I __pycache__ venv/lib/python3.8/site-packages/opentelemetry

venv/lib/python3.8/site-packages/opentelemetry
├── configuration                                                                                                                                                                                      │   ├── __init__.py
│   └── py.typed                                                                                                                                                                                       ├── context
│   ├── aiocontextvarsfix.py                                                                                                                                                                           │   ├── context.py
│   ├── contextvars_context.py
│   ├── __init__.py
│   ├── py.typed
│   └── threadlocal_context.py
├── correlationcontext
│   ├── __init__.py
│   └── propagation
│       └── __init__.py
├── distributedcontext
│   └── py.typed
├── metrics
│   ├── __init__.py
│   └── py.typed
├── propagators
│   ├── composite.py
│   └── __init__.py
├── sdk
│   ├── metrics
│   │   ├── export
│   │   │   ├── aggregate.py
│   │   │   ├── batcher.py
│   │   │   ├── controller.py
│   │   │   ├── __init__.py                                                                                                                                                                            │   │   │   └── in_memory_metrics_exporter.py
│   │   ├── __init__.py
│   │   └── view.py
│   ├── resources
│   │   └── __init__.py
│   ├── trace
│   │   ├── export
│   │   │   ├── __init__.py
│   │   │   └── in_memory_span_exporter.py
│   │   ├── __init__.py
│   │   └── propagation
│   │       ├── b3_format.py
│   │       └── __init__.py
│   └── util
│       ├── __init__.py
│       └── instrumentation.py
├── trace
│   ├── __init__.py
│   ├── propagation
│   │   ├── httptextformat.py
│   │   ├── __init__.py
│   │   └── tracecontexthttptextformat.py
│   ├── py.typed
│   ├── sampling.py
│   ├── span.py
│   └── status.py
└── util
    ├── __init__.py
    ├── py.typed
    └── types.py

18 directories, 41 files

Is this by design? It is included in the package resources so this still works:

>>> import pkg_resources
>>> pkg_resources.get_distribution("opentelemetry-sdk").version
'0.12b0'