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

openlit/instrumentation/pinecone/__init__.py

@@ -10,9 +10,37 @@ from openlit.instrumentation.pinecone.async_pinecone import async_general_wrap
 
 _instruments = ("pinecone >= 7.3.0",)
 
+# Pinecone sync operations
+PINECONE_SYNC_OPERATIONS = [
+    ("pinecone.pinecone", "Pinecone.create_index", "pinecone.create_collection"),
+    ("pinecone.pinecone", "Pinecone.create_index_for_model", "pinecone.create_collection"),
+    ("pinecone.db_data.index", "Index.upsert", "pinecone.upsert"),
+    ("pinecone.db_data.index", "Index.upsert_records", "pinecone.upsert_records"),
+    ("pinecone.db_data.index", "Index.query", "pinecone.query"),
+    ("pinecone.db_data.index", "Index.search", "pinecone.search"),
+    ("pinecone.db_data.index", "Index.fetch", "pinecone.fetch"),
+    ("pinecone.db_data.index", "Index.search_records", "pinecone.search_records"),
+    ("pinecone.db_data.index", "Index.update", "pinecone.update"),
+    ("pinecone.db_data.index", "Index.delete", "pinecone.delete"),
+]
+
+# Pinecone async operations
+PINECONE_ASYNC_OPERATIONS = [
+    ("pinecone.pinecone_asyncio", "PineconeAsyncio.create_index", "pinecone.create_index"),
+    ("pinecone.pinecone_asyncio", "PineconeAsyncio.create_index_for_model", "pinecone.create_index"),
+    ("pinecone.db_data.index_asyncio", "_IndexAsyncio.upsert", "pinecone.upsert"),
+    ("pinecone.db_data.index_asyncio", "_IndexAsyncio.upsert_records", "pinecone.upsert_records"),
+    ("pinecone.db_data.index_asyncio", "_IndexAsyncio.query", "pinecone.query"),
+    ("pinecone.db_data.index_asyncio", "_IndexAsyncio.search", "pinecone.search"),
+    ("pinecone.db_data.index_asyncio", "_IndexAsyncio.fetch", "pinecone.fetch"),
+    ("pinecone.db_data.index_asyncio", "_IndexAsyncio.search_records", "pinecone.search_records"),
+    ("pinecone.db_data.index_asyncio", "_IndexAsyncio.update", "pinecone.update"),
+    ("pinecone.db_data.index_asyncio", "_IndexAsyncio.delete", "pinecone.delete"),
+]
+
 class PineconeInstrumentor(BaseInstrumentor):
     """
-    An instrumentor for Pinecone's client library.
+    An instrumentor for Pinecone client library.
     """
 
     def instrumentation_dependencies(self) -> Collection[str]:
@@ -28,147 +56,23 @@ class PineconeInstrumentor(BaseInstrumentor):
         metrics = kwargs.get("metrics_dict")
         disable_metrics = kwargs.get("disable_metrics")
 
-        # Sync operations
-        wrap_function_wrapper(
-            "pinecone.pinecone",
-            "Pinecone.create_index",
-            general_wrap("pinecone.create_collection", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.pinecone",
-            "Pinecone.create_index_for_model",
-            general_wrap("pinecone.create_collection", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index",
-            "Index.upsert",
-            general_wrap("pinecone.upsert", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index",
-            "Index.upsert_records",
-            general_wrap("pinecone.upsert_records", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index",
-            "Index.query",
-            general_wrap("pinecone.query", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index",
-            "Index.search",
-            general_wrap("pinecone.search", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index",
-            "Index.fetch",
-            general_wrap("pinecone.fetch", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index",
-            "Index.search_records",
-            general_wrap("pinecone.search_records", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index",
-            "Index.update",
-            general_wrap("pinecone.update", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index",
-            "Index.delete",
-            general_wrap("pinecone.delete", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        # Async operations
-        wrap_function_wrapper(
-            "pinecone.pinecone_asyncio",
-            "PineconeAsyncio.create_index",
-            async_general_wrap("pinecone.create_index", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.pinecone_asyncio",
-            "PineconeAsyncio.create_index_for_model",
-            async_general_wrap("pinecone.create_index", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index_asyncio",
-            "_IndexAsyncio.upsert",
-            async_general_wrap("pinecone.upsert", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index_asyncio",
-            "_IndexAsyncio.upsert_records",
-            async_general_wrap("pinecone.upsert_records", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index_asyncio",
-            "_IndexAsyncio.query",
-            async_general_wrap("pinecone.query", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index_asyncio",
-            "_IndexAsyncio.search",
-            async_general_wrap("pinecone.search", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index_asyncio",
-            "_IndexAsyncio.fetch",
-            async_general_wrap("pinecone.fetch", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index_asyncio",
-            "_IndexAsyncio.search_records",
-            async_general_wrap("pinecone.search_records", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index_asyncio",
-            "_IndexAsyncio.update",
-            async_general_wrap("pinecone.update", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
-
-        wrap_function_wrapper(
-            "pinecone.db_data.index_asyncio",
-            "_IndexAsyncio.delete",
-            async_general_wrap("pinecone.delete", version, environment, application_name,
-                tracer, pricing_info, capture_message_content, metrics, disable_metrics),
-        )
+        # Wrap sync operations
+        for module, class_method, endpoint in PINECONE_SYNC_OPERATIONS:
+            wrap_function_wrapper(
+                module,
+                class_method,
+                general_wrap(endpoint, version, environment, application_name,
+                    tracer, pricing_info, capture_message_content, metrics, disable_metrics),
+            )
+
+        # Wrap async operations
+        for module, class_method, endpoint in PINECONE_ASYNC_OPERATIONS:
+            wrap_function_wrapper(
+                module,
+                class_method,
+                async_general_wrap(endpoint, version, environment, application_name,
+                    tracer, pricing_info, capture_message_content, metrics, disable_metrics),
+            )
 
     def _uninstrument(self, **kwargs):
         pass

openlit/instrumentation/pinecone/async_pinecone.py

@@ -5,13 +5,11 @@ Module for monitoring Pinecone async API calls.
 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.__helpers import handle_exception
 from openlit.instrumentation.pinecone.utils import (
     process_vectordb_response,
     DB_OPERATION_MAP,
+    set_server_address_and_port,
 )
 
 def async_general_wrap(gen_ai_endpoint, version, environment, application_name,
@@ -29,7 +27,7 @@ def async_general_wrap(gen_ai_endpoint, version, environment, application_name,
             return await wrapped(*args, **kwargs)
 
         # Get server address and port using the standard helper
-        server_address, server_port = set_server_address_and_port(instance, "pinecone.io", 443)
+        server_address, server_port = set_server_address_and_port(instance)
 
         db_operation = DB_OPERATION_MAP.get(gen_ai_endpoint, "unknown")
         if db_operation == "create_collection":
@@ -39,10 +37,10 @@ def async_general_wrap(gen_ai_endpoint, version, environment, application_name,
         span_name = f"{db_operation} {namespace}"
 
         with tracer.start_as_current_span(span_name, kind=SpanKind.CLIENT) as span:
-            try:
-                start_time = time.time()
-                response = await wrapped(*args, **kwargs)
+            start_time = time.time()
+            response = await wrapped(*args, **kwargs)
 
+            try:
                 # Process response and generate telemetry
                 response = process_vectordb_response(
                     response, db_operation, server_address, server_port,
@@ -50,10 +48,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

@@ -5,13 +5,11 @@ Module for monitoring Pinecone API calls.
 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.__helpers import handle_exception
 from openlit.instrumentation.pinecone.utils import (
     process_vectordb_response,
     DB_OPERATION_MAP,
+    set_server_address_and_port,
 )
 
 def general_wrap(gen_ai_endpoint, version, environment, application_name,
@@ -29,7 +27,7 @@ def general_wrap(gen_ai_endpoint, version, environment, application_name,
             return wrapped(*args, **kwargs)
 
         # Get server address and port using the standard helper
-        server_address, server_port = set_server_address_and_port(instance, "pinecone.io", 443)
+        server_address, server_port = set_server_address_and_port(instance)
 
         db_operation = DB_OPERATION_MAP.get(gen_ai_endpoint, "unknown")
         if db_operation == "create_collection":
@@ -39,10 +37,10 @@ def general_wrap(gen_ai_endpoint, version, environment, application_name,
         span_name = f"{db_operation} {namespace}"
 
         with tracer.start_as_current_span(span_name, kind=SpanKind.CLIENT) as span:
-            try:
-                start_time = time.time()
-                response = wrapped(*args, **kwargs)
+            start_time = time.time()
+            response = wrapped(*args, **kwargs)
 
+            try:
                 # Process response and generate telemetry
                 response = process_vectordb_response(
                     response, db_operation, server_address, server_port,
@@ -50,10 +48,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

@@ -2,7 +2,7 @@
 Pinecone OpenTelemetry instrumentation utility functions
 """
 import time
-
+from urllib.parse import urlparse
 from opentelemetry.trace import Status, StatusCode
 
 from openlit.__helpers import (
@@ -30,6 +30,46 @@ def object_count(obj):
     """
     return len(obj) if obj else 0
 
+def set_server_address_and_port(instance):
+    """
+    Extracts server address and port from Pinecone client instance.
+    
+    Args:
+        instance: Pinecone client instance
+        
+    Returns:
+        tuple: (server_address, server_port)
+    """
+    server_address = "pinecone.io"
+    server_port = 443
+
+    # Try getting base_url from multiple potential attributes
+    base_client = getattr(instance, "_client", None)
+    base_url = getattr(base_client, "base_url", None)
+
+    if not base_url:
+        # Attempt to get host from instance.config.host (used by Pinecone)
+        config = getattr(instance, "config", None)
+        base_url = getattr(config, "host", None)
+
+    if base_url:
+        if isinstance(base_url, str):
+            # Check if its a full URL or just a hostname
+            if base_url.startswith(("http://", "https://")):
+                try:
+                    url = urlparse(base_url)
+                    if url.hostname:
+                        server_address = url.hostname
+                    if url.port:
+                        server_port = url.port
+                except Exception:
+                    pass
+            else:
+                # Just a hostname
+                server_address = base_url
+
+    return server_address, server_port
+
 def common_vectordb_logic(scope, environment, application_name,
     metrics, capture_message_content, disable_metrics, version, instance=None):
     """
@@ -72,7 +112,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 +144,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: