[Feature Request]: Don't import posthog stuff unless anonymized telemetry is enabled

[beggers]

Describe the problem

Right now we import the necessary libs for product telemetry even if it's disabled.

Describe the proposed solution

Don't import the necessary libs for product telemetry when it's disabled.

Alternatives considered

No response

Importance

nice to have

Additional Information

No response

[mmangione]

I agree here. We are seeing crashes in the Python client even with telemetry disabled on the server and in the client start up. This is aggravating and should run the opposite direction.

Python 3.10.12 (main, Nov 20 2023, 15:14:05) [GCC 11.4.0]
Type 'copyright', 'credits' or 'license' for more information
IPython 8.19.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: from chromadb import HttpClient
   ...: from chromadb.config import Settings
   ...: 
   ...: settings = Settings(chroma_api_impl="chromadb.api.fastapi.FastAPI", allow_reset=True, anonymized_telemetry=Fa
   ...: lse)
   ...: client = HttpClient(host='localhost', port=8000, settings=settings)
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
Cell In[1], line 5
      2 from chromadb.config import Settings
      4 settings = Settings(chroma_api_impl="chromadb.api.fastapi.FastAPI", allow_reset=True, anonymized_telemetry=False)
----> 5 client = HttpClient(host='localhost', port=8000, settings=settings)

File ~/chroma/chroma_env/lib/python3.10/site-packages/chromadb/__init__.py:181, in HttpClient(host, port, ssl, headers, settings, tenant, database)
    178 settings.chroma_server_ssl_enabled = ssl
    179 settings.chroma_server_headers = headers
--> 181 return ClientCreator(tenant=tenant, database=database, settings=settings)

File ~/chroma/chroma_env/lib/python3.10/site-packages/chromadb/api/client.py:138, in Client.__init__(self, tenant, database, settings)
    132 def __init__(
    133     self,
    134     tenant: str = DEFAULT_TENANT,
    135     database: str = DEFAULT_DATABASE,
    136     settings: Settings = Settings(),
    137 ) -> None:
--> 138     super().__init__(settings=settings)
    139     self.tenant = tenant
    140     self.database = database

File ~/chroma/chroma_env/lib/python3.10/site-packages/chromadb/api/client.py:41, in SharedSystemClient.__init__(self, settings)
     36 def __init__(
     37     self,
     38     settings: Settings = Settings(),
     39 ) -> None:
     40     self._identifier = SharedSystemClient._get_identifier_from_settings(settings)
---> 41     SharedSystemClient._create_system_if_not_exists(self._identifier, settings)

File ~/chroma/chroma_env/lib/python3.10/site-packages/chromadb/api/client.py:52, in SharedSystemClient._create_system_if_not_exists(cls, identifier, settings)
     49     cls._identifer_to_system[identifier] = new_system
     51     new_system.instance(ProductTelemetryClient)
---> 52     new_system.instance(ServerAPI)
     54     new_system.start()
     55 else:

File ~/chroma/chroma_env/lib/python3.10/site-packages/chromadb/config.py:360, in System.instance(self, type)
    358     key = _abstract_type_keys[type_fqn]
    359     fqn = self.settings.require(key)
--> 360     type = get_class(fqn, type)
    362 if type not in self._instances:
    363     impl = type(self)

File ~/chroma/chroma_env/lib/python3.10/site-packages/chromadb/config.py:409, in get_class(fqn, type)
    407 """Given a fully qualifed class name, import the module and return the class"""
    408 module_name, class_name = fqn.rsplit(".", 1)
--> 409 module = importlib.import_module(module_name)
    410 cls = getattr(module, class_name)
    411 return cast(Type[C], cls)

File /usr/lib/python3.10/importlib/__init__.py:126, in import_module(name, package)
    124             break
    125         level += 1
--> 126 return _bootstrap._gcd_import(name[level:], package, level)

File <frozen importlib._bootstrap>:1050, in _gcd_import(name, package, level)

File <frozen importlib._bootstrap>:1027, in _find_and_load(name, import_)

File <frozen importlib._bootstrap>:1006, in _find_and_load_unlocked(name, import_)

File <frozen importlib._bootstrap>:688, in _load_unlocked(spec)

File <frozen importlib._bootstrap_external>:883, in exec_module(self, module)

File <frozen importlib._bootstrap>:241, in _call_with_frames_removed(f, *args, **kwds)

File ~/chroma/chroma_env/lib/python3.10/site-packages/chromadb/api/fastapi.py:36
     15 from chromadb.api.types import (
     16     DataLoader,
     17     Documents,
   (...)
     31     validate_batch,
     32 )
     33 from chromadb.auth import (
     34     ClientAuthProvider,
     35 )
---> 36 from chromadb.auth.providers import RequestsClientAuthProtocolAdapter
     37 from chromadb.auth.registry import resolve_provider
     38 from chromadb.config import DEFAULT_DATABASE, DEFAULT_TENANT, Settings, System

File ~/chroma/chroma_env/lib/python3.10/site-packages/chromadb/auth/providers.py:19
     17 from chromadb.auth.registry import register_provider, resolve_provider
     18 from chromadb.config import System
---> 19 from chromadb.telemetry.opentelemetry import (
     20     OpenTelemetryGranularity,
     21     trace_method,
     22 )
     24 T = TypeVar("T")
     26 logger = logging.getLogger(__name__)

File ~/chroma/chroma_env/lib/python3.10/site-packages/chromadb/telemetry/opentelemetry/__init__.py:5
      2 from enum import Enum
      3 from typing import Any, Callable, Dict, Optional, Sequence, Union
----> 5 from opentelemetry import trace
      6 from opentelemetry.sdk.resources import SERVICE_NAME, Resource
      7 from opentelemetry.sdk.trace import TracerProvider

ModuleNotFoundError: No module named 'opentelemetry'

[beggers]

hey @mmangione I see that this is a different issue and you've opened a bug for it. I'll look into it there.