Add log message when fsspec cannot be imported.. (#4037)

DylanRussell · web-flow · commit 51db6ad4339d · 2025-12-29T15:56:36.000+01:00
* Add log message

* Add changelog, adjust err msg
diff --git a/util/opentelemetry-util-genai/CHANGELOG.md b/util/opentelemetry-util-genai/CHANGELOG.md
@@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## Unreleased
 
+- Log error when `fsspec` fails to be imported instead of silently failing ([#4037](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/4037)).
 - Minor change to check LRU cache in Completion Hook before acquiring semaphore/thread ([#3907](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3907)).
 - Add environment variable for genai upload hook queue size
   ([https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3943](#3943))
diff --git a/util/opentelemetry-util-genai/src/opentelemetry/util/genai/_upload/__init__.py b/util/opentelemetry-util-genai/src/opentelemetry/util/genai/_upload/__init__.py
@@ -49,6 +49,9 @@ def upload_completion_hook() -> CompletionHook:
             UploadCompletionHook,
         )
     except ImportError:
+        _logger.exception(
+            "Failed to import `fsspec`. Falling back to `NoOpCompletionHook`."
+        )
         return _NoOpCompletionHook()
 
     environ_max_queue_size = environ.get(

Original file line number	Diff line number	Diff line change
`@@ -49,6 +49,9 @@ def upload_completion_hook() -> CompletionHook:`
`49`	`49`	`UploadCompletionHook,`
`50`	`50`	`)`
`51`	`51`	`except ImportError:`
	`52`	`+ _logger.exception(`
	`53`	+ "Failed to import `fsspec`. Falling back to `NoOpCompletionHook`."
	`54`	`+ )`
`52`	`55`	`return _NoOpCompletionHook()`
`53`	`56`
`54`	`57`	`environ_max_queue_size = environ.get(`