openlit 1.34.24__py3-none-any.whl → 1.34.25__py3-none-any.whl

openlit/instrumentation/chroma/__init__.py

@@ -1,5 +1,7 @@
-# pylint: disable=useless-return, bad-staticmethod-argument, disable=duplicate-code
-"""Initializer of Auto Instrumentation of Chromadb Functions"""
+"""
+OpenLIT ChromaDB Instrumentation
+"""
+
 from typing import Collection
 import importlib.metadata
 from opentelemetry.instrumentation.instrumentor import BaseInstrumentor
@@ -10,77 +12,79 @@ from openlit.instrumentation.chroma.chroma import general_wrap
 _instruments = ("chromadb >= 0.4.0",)
 
 class ChromaInstrumentor(BaseInstrumentor):
-    """An instrumentor for Chromadb's client library."""
+    """
+    An instrumentor for ChromaDB's client library.
+    """
 
     def instrumentation_dependencies(self) -> Collection[str]:
         return _instruments
 
     def _instrument(self, **kwargs):
-        application_name = kwargs.get("application_name")
-        environment = kwargs.get("environment")
+        version = importlib.metadata.version("chromadb")
+        environment = kwargs.get("environment", "default")
+        application_name = kwargs.get("application_name", "default")
         tracer = kwargs.get("tracer")
+        pricing_info = kwargs.get("pricing_info", {})
+        capture_message_content = kwargs.get("capture_message_content", False)
         metrics = kwargs.get("metrics_dict")
-        pricing_info = kwargs.get("pricing_info")
-        capture_message_content = kwargs.get("capture_message_content")
         disable_metrics = kwargs.get("disable_metrics")
-        version = importlib.metadata.version("chromadb")
 
+        # Sync operations
         wrap_function_wrapper(
-            "chromadb.db",  
-            "DB.create_collection",  
+            "chromadb.db",
+            "DB.create_collection",
             general_wrap("chroma.create_collection", version, environment, application_name,
-                 tracer, pricing_info, capture_message_content, metrics, disable_metrics),
+                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
         )
 
         wrap_function_wrapper(
-            "chromadb",  
-            "Collection.add",  
+            "chromadb",
+            "Collection.add",
             general_wrap("chroma.add", version, environment, application_name,
-                 tracer, pricing_info, capture_message_content, metrics, disable_metrics),
+                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
         )
 
         wrap_function_wrapper(
-            "chromadb",  
-            "Collection.get",  
+            "chromadb",
+            "Collection.get",
             general_wrap("chroma.get", version, environment, application_name,
-                 tracer, pricing_info, capture_message_content, metrics, disable_metrics),
+                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
         )
 
         wrap_function_wrapper(
-            "chromadb",  
-            "Collection.peek",  
+            "chromadb",
+            "Collection.peek",
             general_wrap("chroma.peek", version, environment, application_name,
-                 tracer, pricing_info, capture_message_content, metrics, disable_metrics),
+                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
         )
 
         wrap_function_wrapper(
-            "chromadb",  
-            "Collection.query",  
+            "chromadb",
+            "Collection.query",
             general_wrap("chroma.query", version, environment, application_name,
-                 tracer, pricing_info, capture_message_content, metrics, disable_metrics),
+                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
         )
 
         wrap_function_wrapper(
-            "chromadb",  
-            "Collection.update",  
+            "chromadb",
+            "Collection.update",
             general_wrap("chroma.update", version, environment, application_name,
-                 tracer, pricing_info, capture_message_content, metrics, disable_metrics),
+                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
         )
 
         wrap_function_wrapper(
-            "chromadb",  
-            "Collection.upsert",  
+            "chromadb",
+            "Collection.upsert",
             general_wrap("chroma.upsert", version, environment, application_name,
-                 tracer, pricing_info, capture_message_content, metrics, disable_metrics),
+                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
         )
+
         wrap_function_wrapper(
-            "chromadb",  
-            "Collection.delete",  
+            "chromadb",
+            "Collection.delete",
             general_wrap("chroma.delete", version, environment, application_name,
-                 tracer, pricing_info, capture_message_content, metrics, disable_metrics),
+                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
         )
 
-
-    @staticmethod
     def _uninstrument(self, **kwargs):
         pass

openlit/instrumentation/chroma/chroma.py

@@ -1,199 +1,59 @@
-# pylint: disable=duplicate-code, broad-exception-caught, too-many-statements, unused-argument, possibly-used-before-assignment
 """
-Module for monitoring ChromaDB.
+Module for monitoring ChromaDB API calls.
 """
 
-import logging
-from opentelemetry.trace import SpanKind, Status, StatusCode
-from opentelemetry.sdk.resources import SERVICE_NAME, TELEMETRY_SDK_NAME, DEPLOYMENT_ENVIRONMENT
-from openlit.__helpers import handle_exception
-from openlit.semcov import SemanticConvention
-
-# Initialize logger for logging potential issues and operations
-logger = logging.getLogger(__name__)
-
-def object_count(obj):
-    """
-    Counts Length of object if it exists, Else returns None
-    """
-    try:
-        cnt = len(obj)
-    # pylint: disable=bare-except
-    except:
-        cnt = 0
-
-    return cnt
+import time
+from opentelemetry.trace import SpanKind
+from opentelemetry import context as context_api
+from openlit.__helpers import (
+    handle_exception,
+    set_server_address_and_port,
+)
+from openlit.instrumentation.chroma.utils import (
+    process_vectordb_response,
+    DB_OPERATION_MAP,
+)
 
 def general_wrap(gen_ai_endpoint, version, environment, application_name,
-                 tracer, pricing_info, capture_message_content, metrics, disable_metrics):
+    tracer, pricing_info, capture_message_content, metrics, disable_metrics):
     """
-    Creates a wrapper around a function call to trace and log its execution metrics.
-
-    This function wraps any given function to measure its execution time,
-    log its operation, and trace its execution using OpenTelemetry.
-    
-    Parameters:
-    - gen_ai_endpoint (str): A descriptor or name for the endpoint being traced.
-    - version (str): The version of the Langchain application.
-    - environment (str): The deployment environment (e.g., 'production', 'development').
-    - application_name (str): Name of the Langchain application.
-    - tracer (opentelemetry.trace.Tracer): The tracer object used for OpenTelemetry tracing.
-    - pricing_info (dict): Information about the pricing for internal metrics (currently not used).
-    - capture_message_content (bool): Flag indicating whether to trace the content of the response.
-
-    Returns:
-    - function: A higher-order function that takes a function 'wrapped' and returns
-                a new function that wraps 'wrapped' with additional tracing and logging.
+    Generates a telemetry wrapper for ChromaDB function calls.
     """
 
     def wrapper(wrapped, instance, args, kwargs):
         """
-        An inner wrapper function that executes the wrapped function, measures execution
-        time, and records trace data using OpenTelemetry.
-
-        Parameters:
-        - wrapped (Callable): The original function that this wrapper will execute.
-        - instance (object): The instance to which the wrapped function belongs. This
-                             is used for instance methods. For static and classmethods,
-                             this may be None.
-        - args (tuple): Positional arguments passed to the wrapped function.
-        - kwargs (dict): Keyword arguments passed to the wrapped function.
-
-        Returns:
-        - The result of the wrapped function call.
-        
-        The wrapper initiates a span with the provided tracer, sets various attributes
-        on the span based on the function's execution and response, and ensures
-        errors are handled and logged appropriately.
+        Wraps the ChromaDB function call.
         """
-        with tracer.start_as_current_span(gen_ai_endpoint, kind= SpanKind.CLIENT) as span:
-            response = wrapped(*args, **kwargs)
 
-            try:
-                span.set_attribute(TELEMETRY_SDK_NAME, "openlit")
-                span.set_attribute(SemanticConvention.GEN_AI_ENDPOINT,
-                                   gen_ai_endpoint)
-                span.set_attribute(DEPLOYMENT_ENVIRONMENT,
-                                   environment)
-                span.set_attribute(SERVICE_NAME,
-                                   application_name)
-                span.set_attribute(SemanticConvention.GEN_AI_OPERATION,
-                                   SemanticConvention.GEN_AI_OPERATION_TYPE_VECTORDB)
-                span.set_attribute(SemanticConvention.DB_SYSTEM_NAME,
-                                   SemanticConvention.DB_SYSTEM_CHROMA)
-                span.set_attribute(SemanticConvention.DB_COLLECTION_NAME,
-                                   instance.name)
+        if context_api.get_value(context_api._SUPPRESS_INSTRUMENTATION_KEY):
+            return wrapped(*args, **kwargs)
 
-                if gen_ai_endpoint == "chroma.add":
-                    db_operation = SemanticConvention.DB_OPERATION_ADD
-                    span.set_attribute(SemanticConvention.DB_OPERATION_NAME,
-                                       SemanticConvention.DB_OPERATION_ADD)
-                    span.set_attribute(SemanticConvention.DB_ID_COUNT,
-                                       object_count(kwargs.get("ids", [])))
-                    span.set_attribute(SemanticConvention.DB_VECTOR_COUNT,
-                                       object_count(kwargs.get("embeddings", [])))
-                    span.set_attribute(SemanticConvention.DB_VECTOR_COUNT,
-                                       object_count(kwargs.get("metadatas", [])))
-                    span.set_attribute(SemanticConvention.DB_DOCUMENTS_COUNT,
-                                       object_count(kwargs.get("documents", [])))
+        # Get server address and port using the standard helper
+        server_address, server_port = set_server_address_and_port(instance, "localhost", 8000)
 
-                elif gen_ai_endpoint == "chroma.get":
-                    db_operation = SemanticConvention.DB_OPERATION_GET
-                    span.set_attribute(SemanticConvention.DB_OPERATION_NAME,
-                                       SemanticConvention.DB_OPERATION_GET)
-                    span.set_attribute(SemanticConvention.DB_ID_COUNT,
-                                       object_count(kwargs.get("ids", [])))
-                    span.set_attribute(SemanticConvention.DB_QUERY_LIMIT,
-                                       kwargs.get("limit", ""))
-                    span.set_attribute(SemanticConvention.DB_OFFSET,
-                                       kwargs.get("offset", ""))
-                    span.set_attribute(SemanticConvention.DB_WHERE_DOCUMENT,
-                                       str(kwargs.get("where_document", "")))
+        db_operation = DB_OPERATION_MAP.get(gen_ai_endpoint, "unknown")
+        if db_operation == "create_collection":
+            namespace = kwargs.get("name") or (args[0] if args else "unknown")
+        else:
+            namespace = getattr(instance, "name", "unknown")
+        span_name = f"{db_operation} {namespace}"
 
-                elif gen_ai_endpoint == "chroma.query":
-                    db_operation = SemanticConvention.DB_OPERATION_QUERY
-                    span.set_attribute(SemanticConvention.DB_OPERATION_NAME,
-                                       SemanticConvention.DB_OPERATION_QUERY)
-                    span.set_attribute(SemanticConvention.DB_STATEMENT,
-                                       str(kwargs.get("query_texts", "")))
-                    span.set_attribute(SemanticConvention.DB_N_RESULTS,
-                                       kwargs.get("n_results", ""))
-                    span.set_attribute(SemanticConvention.DB_FILTER,
-                                       str(kwargs.get("where", "")))
-                    span.set_attribute(SemanticConvention.DB_WHERE_DOCUMENT,
-                                       str(kwargs.get("where_document", "")))
-
-                elif gen_ai_endpoint == "chroma.update":
-                    db_operation = SemanticConvention.DB_OPERATION_UPDATE
-                    span.set_attribute(SemanticConvention.DB_OPERATION_NAME,
-                                       SemanticConvention.DB_OPERATION_UPDATE)
-                    span.set_attribute(SemanticConvention.DB_VECTOR_COUNT,
-                                       object_count(kwargs.get("embeddings", [])))
-                    span.set_attribute(SemanticConvention.DB_VECTOR_COUNT,
-                                       object_count(kwargs.get("metadatas", [])))
-                    span.set_attribute(SemanticConvention.DB_ID_COUNT,
-                                       object_count(kwargs.get("ids", [])))
-                    span.set_attribute(SemanticConvention.DB_DOCUMENTS_COUNT,
-                                       object_count(kwargs.get("documents", [])))
-
-                elif gen_ai_endpoint == "chroma.upsert":
-                    db_operation = SemanticConvention.DB_OPERATION_UPSERT
-                    span.set_attribute(SemanticConvention.DB_OPERATION_NAME,
-                                       SemanticConvention.DB_OPERATION_UPSERT)
-                    span.set_attribute(SemanticConvention.DB_VECTOR_COUNT,
-                                       object_count(kwargs.get("embeddings", [])))
-                    span.set_attribute(SemanticConvention.DB_VECTOR_COUNT,
-                                       object_count(kwargs.get("metadatas", [])))
-                    span.set_attribute(SemanticConvention.DB_ID_COUNT,
-                                       object_count(kwargs.get("ids", [])))
-                    span.set_attribute(SemanticConvention.DB_DOCUMENTS_COUNT,
-                                       object_count(kwargs.get("documents", [])))
-
-                elif gen_ai_endpoint == "chroma.delete":
-                    db_operation = SemanticConvention.DB_OPERATION_DELETE
-                    span.set_attribute(SemanticConvention.DB_OPERATION_NAME,
-                                       SemanticConvention.DB_OPERATION_DELETE)
-                    span.set_attribute(SemanticConvention.DB_ID_COUNT,
-                                       object_count(kwargs.get("ids", [])))
-                    span.set_attribute(SemanticConvention.DB_FILTER,
-                                       str(kwargs.get("where", "")))
-                    span.set_attribute(SemanticConvention.DB_DELETE_ALL,
-                                       kwargs.get("delete_all", False))
-                    span.set_attribute(SemanticConvention.DB_WHERE_DOCUMENT,
-                                       str(kwargs.get("where_document", "")))
-
-                elif gen_ai_endpoint == "chroma.peek":
-                    db_operation = SemanticConvention.DB_OPERATION_PEEK
-                    span.set_attribute(SemanticConvention.DB_OPERATION_NAME,
-                                       SemanticConvention.DB_OPERATION_PEEK)
-
-                span.set_status(Status(StatusCode.OK))
-
-                if disable_metrics is False:
-                    attributes = {
-                        TELEMETRY_SDK_NAME:
-                            "openlit",
-                        SERVICE_NAME:
-                            application_name,
-                        SemanticConvention.DB_SYSTEM_NAME:
-                            SemanticConvention.DB_SYSTEM_CHROMA,
-                        DEPLOYMENT_ENVIRONMENT:
-                            environment,
-                        SemanticConvention.GEN_AI_OPERATION:
-                            SemanticConvention.GEN_AI_OPERATION_TYPE_VECTORDB,
-                        SemanticConvention.DB_OPERATION_NAME:
-                            db_operation
-                    }
-
-                    metrics["db_requests"].add(1, attributes)
+        with tracer.start_as_current_span(span_name, kind=SpanKind.CLIENT) as span:
+            try:
+                start_time = time.time()
+                response = wrapped(*args, **kwargs)
 
-                return response
+                # Process response and generate telemetry
+                response = process_vectordb_response(
+                    response, db_operation, server_address, server_port,
+                    environment, application_name, metrics, start_time, span,
+                    capture_message_content, disable_metrics, version, instance, args, endpoint=gen_ai_endpoint, **kwargs
+                )
 
             except Exception as e:
                 handle_exception(span, e)
-                logger.error("Error in trace creation: %s", e)
 
-                # Return original response
-                return response
+            return response
+
 
     return wrapper

openlit/instrumentation/chroma/utils.py

@@ -0,0 +1,227 @@
+"""
+ChromaDB OpenTelemetry instrumentation utility functions
+"""
+import time
+
+from opentelemetry.trace import Status, StatusCode
+
+from openlit.__helpers import (
+    common_db_span_attributes,
+    record_db_metrics,
+)
+from openlit.semcov import SemanticConvention
+
+# Operation mapping for simple span naming
+DB_OPERATION_MAP = {
+    "chroma.create_collection": SemanticConvention.DB_OPERATION_CREATE_COLLECTION,
+    "chroma.add": SemanticConvention.DB_OPERATION_INSERT,
+    "chroma.get": SemanticConvention.DB_OPERATION_GET,
+    "chroma.peek": SemanticConvention.DB_OPERATION_PEEK,
+    "chroma.query": SemanticConvention.DB_OPERATION_GET,
+    "chroma.update": SemanticConvention.DB_OPERATION_UPDATE,
+    "chroma.upsert": SemanticConvention.DB_OPERATION_UPSERT,
+    "chroma.delete": SemanticConvention.DB_OPERATION_DELETE,
+}
+
+def object_count(obj):
+    """
+    Counts length of object if it exists, else returns 0.
+    """
+    return len(obj) if obj else 0
+
+def common_vectordb_logic(scope, environment, application_name,
+    metrics, capture_message_content, disable_metrics, version, instance=None, endpoint=None):
+    """
+    Process vector database request and generate telemetry.
+    """
+
+    scope._end_time = time.time()
+
+    # Set common database span attributes using helper
+    common_db_span_attributes(scope, SemanticConvention.DB_SYSTEM_CHROMA, scope._server_address, scope._server_port,
+        environment, application_name, version)
+
+    # Set DB operation specific attributes
+    scope._span.set_attribute(SemanticConvention.DB_OPERATION_NAME, scope._db_operation)
+    scope._span.set_attribute(SemanticConvention.DB_CLIENT_OPERATION_DURATION, scope._end_time - scope._start_time)
+
+    # Set collection name from instance
+    if hasattr(instance, "name"):
+        scope._span.set_attribute(SemanticConvention.DB_COLLECTION_NAME, instance.name)
+
+    # Set Create Collection operation specific attributes
+    if scope._db_operation == SemanticConvention.DB_OPERATION_CREATE_COLLECTION:
+        # Standard database attributes
+        collection_name = scope._kwargs.get("name") or (scope._args[0] if scope._args else "unknown")
+        scope._span.set_attribute(SemanticConvention.DB_COLLECTION_NAME, collection_name)
+
+        # Vector database specific attributes (extensions)
+        metadata = scope._kwargs.get("metadata", {})
+        if metadata:
+            scope._span.set_attribute(SemanticConvention.DB_COLLECTION_METADATA, str(metadata))
+
+    elif scope._db_operation == SemanticConvention.DB_OPERATION_INSERT:
+        collection_name = getattr(instance, "name", "unknown")
+        query = scope._kwargs.get("ids", [])
+
+        # Standard database attributes
+        scope._span.set_attribute(SemanticConvention.DB_QUERY_TEXT, str(query))
+        scope._span.set_attribute(SemanticConvention.DB_COLLECTION_NAME, collection_name)
+        scope._span.set_attribute(SemanticConvention.DB_VECTOR_COUNT, object_count(query))
+
+        # Vector database specific attributes (extensions)
+        scope._span.set_attribute(SemanticConvention.DB_QUERY_SUMMARY,
+            f"{scope._db_operation} {collection_name} "
+            f"ids={query} "
+            f"documents={scope._kwargs.get('documents', [])}")
+
+    elif scope._db_operation == SemanticConvention.DB_OPERATION_GET:
+        collection_name = getattr(instance, "name", "unknown")
+
+        # Handle different GET operations based on endpoint
+        if endpoint == "chroma.get":
+            # Collection.get() - retrieve documents by IDs
+            query = scope._kwargs.get("ids", [])
+
+            # Standard database attributes
+            scope._span.set_attribute(SemanticConvention.DB_QUERY_TEXT, str(query))
+            scope._span.set_attribute(SemanticConvention.DB_COLLECTION_NAME, collection_name)
+            scope._span.set_attribute(SemanticConvention.DB_VECTOR_COUNT, object_count(query))
+
+            # Vector database specific attributes (extensions)
+            scope._span.set_attribute(SemanticConvention.DB_FILTER, str(scope._kwargs.get("where", "")))
+
+            scope._span.set_attribute(SemanticConvention.DB_QUERY_SUMMARY,
+                f"{scope._db_operation} {collection_name} "
+                f"ids={query} "
+                f"limit={scope._kwargs.get('limit', 'None')} "
+                f"offset={scope._kwargs.get('offset', 'None')}")
+
+        elif endpoint == "chroma.query":
+            query_texts = scope._kwargs.get("query_texts", [])
+            query_embeddings = scope._kwargs.get("query_embeddings", [])
+
+            # Create comprehensive query text (can be either embeddings or texts)
+            if query_texts:
+                query_content = f"texts={query_texts}"
+            elif query_embeddings:
+                query_content = f"embeddings={len(query_embeddings) if query_embeddings else 0} vectors"
+            else:
+                query_content = "no query provided"
+
+            # Standard database attributes
+            scope._span.set_attribute(SemanticConvention.DB_QUERY_TEXT, query_content)
+            scope._span.set_attribute(SemanticConvention.DB_COLLECTION_NAME, collection_name)
+
+            # Vector database specific attributes (extensions)
+            scope._span.set_attribute(SemanticConvention.DB_VECTOR_QUERY_TOP_K, scope._kwargs.get("n_results", 10))
+            scope._span.set_attribute(SemanticConvention.DB_FILTER, str(scope._kwargs.get("where", "")))
+
+            # Extract response metrics if available
+            if scope._response:
+                # Get number of results returned
+                if hasattr(scope._response, 'get') and scope._response.get('ids'):
+                    returned_rows = object_count(scope._response['ids'][0]) if scope._response['ids'] else 0
+                    scope._span.set_attribute(SemanticConvention.DB_RESPONSE_RETURNED_ROWS, returned_rows)
+
+            scope._span.set_attribute(SemanticConvention.DB_QUERY_SUMMARY,
+                f"{scope._db_operation} {collection_name} "
+                f"n_results={scope._kwargs.get('n_results', 10)} "
+                f"{query_content} "
+                f"filter={scope._kwargs.get('where', 'None')}")
+
+    elif scope._db_operation == SemanticConvention.DB_OPERATION_UPDATE:
+        collection_name = getattr(instance, "name", "unknown")
+        query = scope._kwargs.get("ids", [])
+
+        # Standard database attributes
+        scope._span.set_attribute(SemanticConvention.DB_QUERY_TEXT, str(query))
+        scope._span.set_attribute(SemanticConvention.DB_COLLECTION_NAME, collection_name)
+        scope._span.set_attribute(SemanticConvention.DB_VECTOR_COUNT, object_count(query))
+
+        # Vector database specific attributes (extensions)
+        scope._span.set_attribute(SemanticConvention.DB_QUERY_SUMMARY,
+            f"{scope._db_operation} {collection_name} "
+            f"ids={query} "
+            f"embeddings={scope._kwargs.get('embeddings', 'None')} "
+            f"metadatas={scope._kwargs.get('metadatas', 'None')} "
+            f"documents={scope._kwargs.get('documents', 'None')}")
+
+    elif scope._db_operation == SemanticConvention.DB_OPERATION_UPSERT:
+        collection_name = getattr(instance, "name", "unknown")
+        query = scope._kwargs.get("ids", [])
+
+        # Standard database attributes
+        scope._span.set_attribute(SemanticConvention.DB_QUERY_TEXT, str(query))
+        scope._span.set_attribute(SemanticConvention.DB_COLLECTION_NAME, collection_name)
+        scope._span.set_attribute(SemanticConvention.DB_VECTOR_COUNT, object_count(query))
+
+        # Vector database specific attributes (extensions)
+        scope._span.set_attribute(SemanticConvention.DB_QUERY_SUMMARY,
+            f"{scope._db_operation} {collection_name} "
+            f"ids={query} "
+            f"embeddings={scope._kwargs.get('embeddings', 'None')} "
+            f"metadatas={scope._kwargs.get('metadatas', 'None')} "
+            f"documents={scope._kwargs.get('documents', 'None')}")
+
+    elif scope._db_operation == SemanticConvention.DB_OPERATION_DELETE:
+        collection_name = getattr(instance, "name", "unknown")
+        query = scope._kwargs.get("ids", [])
+
+        # Standard database attributes
+        scope._span.set_attribute(SemanticConvention.DB_QUERY_TEXT, str(query))
+        scope._span.set_attribute(SemanticConvention.DB_COLLECTION_NAME, collection_name)
+        scope._span.set_attribute(SemanticConvention.DB_VECTOR_COUNT, object_count(query))
+
+        # Vector database specific attributes (extensions)
+        scope._span.set_attribute(SemanticConvention.DB_FILTER, str(scope._kwargs.get("where", "")))
+        scope._span.set_attribute(SemanticConvention.DB_QUERY_SUMMARY,
+            f"{scope._db_operation} {collection_name} "
+            f"ids={query} "
+            f"filter={scope._kwargs.get('where', 'None')} "
+            f"delete_all={scope._kwargs.get('delete_all', False)}")
+
+    elif scope._db_operation == SemanticConvention.DB_OPERATION_PEEK:
+        collection_name = getattr(instance, "name", "unknown")
+        query = f"PEEK limit={scope._kwargs.get('limit', '')}"
+
+        # Standard database attributes
+        scope._span.set_attribute(SemanticConvention.DB_QUERY_TEXT, query)
+        scope._span.set_attribute(SemanticConvention.DB_COLLECTION_NAME, collection_name)
+        scope._span.set_attribute(SemanticConvention.DB_VECTOR_COUNT, scope._kwargs.get("limit", ""))
+
+        # Vector database specific attributes (extensions)
+        scope._span.set_attribute(SemanticConvention.DB_QUERY_SUMMARY,
+            f"{scope._db_operation} {collection_name} "
+            f"limit={scope._kwargs.get('limit', 'None')}")
+
+    scope._span.set_status(Status(StatusCode.OK))
+
+    # Record metrics using helper
+    if not disable_metrics:
+        record_db_metrics(metrics, SemanticConvention.DB_SYSTEM_CHROMA, scope._server_address, scope._server_port,
+            environment, application_name, scope._start_time, scope._end_time)
+
+def process_vectordb_response(response, db_operation, server_address, server_port,
+    environment, application_name, metrics, start_time, span,
+    capture_message_content=False, disable_metrics=False,
+    version="1.0.0", instance=None, args=None, endpoint=None, **kwargs):
+    """
+    Process vector database response and generate telemetry following OpenTelemetry conventions.
+    """
+
+    scope = type("GenericScope", (), {})()
+
+    scope._start_time = start_time
+    scope._span = span
+    scope._kwargs = kwargs
+    scope._args = args or []
+    scope._db_operation = db_operation
+    scope._response = response
+    scope._server_address = server_address
+    scope._server_port = server_port
+
+    common_vectordb_logic(scope, environment, application_name,
+        metrics, capture_message_content, disable_metrics, version, instance, endpoint)
+
+    return response

openlit/instrumentation/pinecone/async_pinecone.py

@@ -50,10 +50,9 @@ def async_general_wrap(gen_ai_endpoint, version, environment, application_name,
                     capture_message_content, disable_metrics, version, instance, args, **kwargs
                 )
 
-                return response
-
             except Exception as e:
                 handle_exception(span, e)
-                raise
+
+            return response
 
     return wrapper

openlit/instrumentation/pinecone/pinecone.py

@@ -50,10 +50,9 @@ def general_wrap(gen_ai_endpoint, version, environment, application_name,
                     capture_message_content, disable_metrics, version, instance, args, **kwargs
                 )
 
-                return response
-
             except Exception as e:
                 handle_exception(span, e)
-                raise
+
+            return response
 
     return wrapper

openlit/instrumentation/pinecone/utils.py

@@ -72,7 +72,10 @@ def common_vectordb_logic(scope, environment, application_name,
         # Vector database specific attributes (extensions)
         scope._span.set_attribute(SemanticConvention.DB_VECTOR_QUERY_TOP_K, query.get("top_k", -1))
         scope._span.set_attribute(SemanticConvention.DB_QUERY_SUMMARY,
-            f"SEARCH {namespace} top_k={query.get('top_k', -1)} text={query_text} vector={query_vector}")
+            f"{scope._db_operation} {namespace} "
+            f"top_k={query.get('top_k', -1)} "
+            f"text={query_text} "
+            f"vector={query_vector}")
 
     elif scope._db_operation == SemanticConvention.DB_OPERATION_QUERY:
         namespace = scope._kwargs.get("namespace", "default") or (scope._args[0] if scope._args else "unknown")
@@ -101,7 +104,8 @@ def common_vectordb_logic(scope, environment, application_name,
 
         # Vector database specific attributes (extensions)
         scope._span.set_attribute(SemanticConvention.DB_QUERY_SUMMARY,
-            f"FETCH {namespace} ids={query}")
+            f"{scope._db_operation} {namespace} "
+            f"ids={query}")
         scope._span.set_attribute(SemanticConvention.DB_RESPONSE_RETURNED_ROWS, object_count(scope._response.vectors))
 
     elif scope._db_operation == SemanticConvention.DB_OPERATION_UPDATE:

{openlit-1.34.24.dist-info → openlit-1.34.25.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: openlit
-Version: 1.34.24
+Version: 1.34.25
 Summary: OpenTelemetry-native Auto instrumentation library for monitoring LLM Applications and GPUs, facilitating the integration of observability into your GenAI-driven projects
 License: Apache-2.0
 Keywords: OpenTelemetry,otel,otlp,llm,tracing,openai,anthropic,claude,cohere,llm monitoring,observability,monitoring,gpt,Generative AI,chatGPT,gpu

{openlit-1.34.24.dist-info → openlit-1.34.25.dist-info}/RECORD

@@ -38,8 +38,9 @@ openlit/instrumentation/azure_ai_inference/utils.py,sha256=JqKZgb6VppDbAQ2RdH_dQ
 openlit/instrumentation/bedrock/__init__.py,sha256=D4Wyjb9kJASUQMy5hI41Mvp_csP4k4F6tASdQjt8IZ0,1681
 openlit/instrumentation/bedrock/bedrock.py,sha256=_2oTa9cf-OxHTFCmNUWT__vHnv7-wUcxm9-6nC1K3jg,8036
 openlit/instrumentation/bedrock/utils.py,sha256=7qn8pI-BQcW8ZLxYJb_W3Yzncqbz4QBCR9f2u9N7C4Y,9345
-openlit/instrumentation/chroma/__init__.py,sha256=4ZeHY1OInRKQbb4qg8BVvGJtWN1XdzW6mosqi7-6ruE,3353
-openlit/instrumentation/chroma/chroma.py,sha256=fPIX6vgcDFkBgKGxrkXZWDaBtZa5y-PV3jJjKsFqjDg,10640
+openlit/instrumentation/chroma/__init__.py,sha256=eSuPdZiuqtThdkSjJ0TdEz3kS0OEIoRN2g9rggSgWgE,3258
+openlit/instrumentation/chroma/chroma.py,sha256=vhPF9yQS1Jz_tEGsXP6rNHlqlsLH1B6ggQwU3FXWcvc,2031
+openlit/instrumentation/chroma/utils.py,sha256=y2SjfUcXpcVp9eggczl6wyDq847sGCIJ6dFkJzVJuPo,10799
 openlit/instrumentation/cohere/__init__.py,sha256=FIJ_QokUZcN9UsVPWckRfHiLVJ5zj3jtcOjQmjQA5f0,2978
 openlit/instrumentation/cohere/async_cohere.py,sha256=wgctJa-BcKEkVguLorR7mYRShv7GiioF1_zxevvzXTw,6990
 openlit/instrumentation/cohere/cohere.py,sha256=7YAj6kwJ1MDLFY3WtbPRehZFIseDVIbvPQJZxG8Qc6A,6814
@@ -118,9 +119,9 @@ openlit/instrumentation/openai_agents/openai_agents.py,sha256=kRWPgjofcOviMi3w7C
 openlit/instrumentation/phidata/__init__.py,sha256=tqls5-UI6FzbjxYgq_qqAfALhWJm8dHn2NtgqiQA4f8,1557
 openlit/instrumentation/phidata/phidata.py,sha256=ohrxs6i0Oik75P2BrjNGbK71tdZg94ZMmaXixrXwV5M,4834
 openlit/instrumentation/pinecone/__init__.py,sha256=DaWkIeG_ukn_JkBfz11TgPjORUUE9vPPjt42dH_26Jk,7210
-openlit/instrumentation/pinecone/async_pinecone.py,sha256=JBQgro4GYQhnp7IMGnk0xmJQJrQZjuVyOwTRc8x3qQU,2098
-openlit/instrumentation/pinecone/pinecone.py,sha256=_Jo_xh44sWDnqv2pA2Jx1ptDol25epJASfxKfHDHrzI,2056
-openlit/instrumentation/pinecone/utils.py,sha256=_Ims5EwmuVd3r2OBg5If2JY7HqsPA_luhmLW9AMsUjw,8995
+openlit/instrumentation/pinecone/async_pinecone.py,sha256=z8jmf69_vqsoFBqm7fUY0dPY2K7q_9oJptkKyE2fbu8,2072
+openlit/instrumentation/pinecone/pinecone.py,sha256=zZenObgPU-LSvN3kbfRU9dwnaG1hUU7j2V6MTY4ahNM,2030
+openlit/instrumentation/pinecone/utils.py,sha256=fvaQnxkyvXw8b18OqvzWHBrT_kxVivQamXL2rOjBrys,9090
 openlit/instrumentation/premai/__init__.py,sha256=3YlqyV-eNA_4aVUHDVUQUvGJRW8iVVcRtREw91yhbyw,1728
 openlit/instrumentation/premai/premai.py,sha256=rWRqfoIZUbTz-M7zgC2Z92gTVv9fCj1Z4iJcsG86YeI,6438
 openlit/instrumentation/premai/utils.py,sha256=K7EKGRDDh1X3OznG4z8H506zzFOHN6MH3oqtxM5eUyM,11409
@@ -152,7 +153,7 @@ openlit/otel/events.py,sha256=VrMjTpvnLtYRBHCiFwJojTQqqNpRCxoD4yJYeQrtPsk,3560
 openlit/otel/metrics.py,sha256=GM2PDloBGRhBTkHHkYaqmOwIAQkY124ZhW4sEqW1Fgk,7086
 openlit/otel/tracing.py,sha256=tjV2bEbEDPUB1Z46gE-UsJsb04sRdFrfbhIDkxViZc0,3103
 openlit/semcov/__init__.py,sha256=9QRSBwOvUS6z7T0HZkE9XlTEnqp-76R_BXM2o4IaEA8,14070
-openlit-1.34.24.dist-info/LICENSE,sha256=xx0jnfkXJvxRnG63LTGOxlggYnIysveWIZ6H3PNdCrQ,11357
-openlit-1.34.24.dist-info/METADATA,sha256=NKUgBOoqkr6scsH-Y8H6DVnjV6cf1Cqx0X5MVx1OPHk,23470
-openlit-1.34.24.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-openlit-1.34.24.dist-info/RECORD,,
+openlit-1.34.25.dist-info/LICENSE,sha256=xx0jnfkXJvxRnG63LTGOxlggYnIysveWIZ6H3PNdCrQ,11357
+openlit-1.34.25.dist-info/METADATA,sha256=R37O_LK3DDYHZybcaM4dTSBEB1sW1nuuK-XbTEyhgFE,23470
+openlit-1.34.25.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+openlit-1.34.25.dist-info/RECORD,,