Fix #2536: Add CREWAI_DISABLE_TELEMETRY environment variable (#2537)

* Fix #2536: Add CREWAI_DISABLE_TELEMETRY environment variable

Co-Authored-By: Joe Moura <joao@crewai.com>

* Fix import order in telemetry test file

Co-Authored-By: Joe Moura <joao@crewai.com>

* Fix telemetry implementation based on PR feedback

Co-Authored-By: Joe Moura <joao@crewai.com>

* Revert telemetry implementation changes while keeping CREWAI_DISABLE_TELEMETRY functionality

Co-Authored-By: Joe Moura <joao@crewai.com>

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: Joe Moura <joao@crewai.com>

docs/telemetry.mdx

@@ -22,7 +22,16 @@ usage of tools, API calls, responses, any data processed by the agents, or secre
 When the `share_crew` feature is enabled, detailed data including task descriptions, agents' backstories or goals, and other specific attributes are collected 
 to provide deeper insights. This expanded data collection may include personal information if users have incorporated it into their crews or tasks. 
 Users should carefully consider the content of their crews and tasks before enabling `share_crew`. 
-Users can disable telemetry by setting the environment variable `OTEL_SDK_DISABLED` to `true`.
+Users can disable telemetry by setting the environment variable `CREWAI_DISABLE_TELEMETRY` to `true` or by setting `OTEL_SDK_DISABLED` to `true` (note that the latter disables all OpenTelemetry instrumentation globally).
+
+### Examples:
+```python
+# Disable CrewAI telemetry only
+os.environ['CREWAI_DISABLE_TELEMETRY'] = 'true'
+
+# Disable all OpenTelemetry (including CrewAI)
+os.environ['OTEL_SDK_DISABLED'] = 'true'
+```
 
 ### Data Explanation:
 | Defaulted | Data                                      | Reason and Specifics                                                                                                       |

src/crewai/telemetry/telemetry.py

@@ -45,10 +45,10 @@ class Telemetry:
     """
 
     def __init__(self):
-        self.ready = False
+        self.ready: bool = False
-        self.trace_set = False
+        self.trace_set: bool = False
 
-        if os.getenv("OTEL_SDK_DISABLED", "false").lower() == "true":
+        if self._is_telemetry_disabled():
             return
 
         try:
@@ -76,6 +76,13 @@ class Telemetry:
                 raise  # Re-raise the exception to not interfere with system signals
             self.ready = False
             
+    def _is_telemetry_disabled(self) -> bool:
+        """Check if telemetry should be disabled based on environment variables."""
+        return (
+            os.getenv("OTEL_SDK_DISABLED", "false").lower() == "true" or
+            os.getenv("CREWAI_DISABLE_TELEMETRY", "false").lower() == "true"
+        )
+
     def set_tracer(self):
         if self.ready and not self.trace_set:
             try:

tests/telemetry/test_telemetry_disable.py

@@ -0,0 +1,30 @@
+import os
+from unittest.mock import patch
+
+import pytest
+
+from crewai.telemetry import Telemetry
+
+
+@pytest.mark.parametrize("env_var,value,expected_ready", [
+    ("OTEL_SDK_DISABLED", "true", False),
+    ("OTEL_SDK_DISABLED", "TRUE", False),
+    ("CREWAI_DISABLE_TELEMETRY", "true", False),
+    ("CREWAI_DISABLE_TELEMETRY", "TRUE", False),
+    ("OTEL_SDK_DISABLED", "false", True),
+    ("CREWAI_DISABLE_TELEMETRY", "false", True),
+])
+def test_telemetry_environment_variables(env_var, value, expected_ready):
+    """Test telemetry state with different environment variable configurations."""
+    with patch.dict(os.environ, {env_var: value}):
+        with patch("crewai.telemetry.telemetry.TracerProvider"):
+            telemetry = Telemetry()
+            assert telemetry.ready is expected_ready
+
+
+def test_telemetry_enabled_by_default():
+    """Test that telemetry is enabled by default."""
+    with patch.dict(os.environ, {}, clear=True):
+        with patch("crewai.telemetry.telemetry.TracerProvider"):
+            telemetry = Telemetry()
+            assert telemetry.ready is True