crewAI
crewAI copied to clipboard
Enable telemetry opt-out #254
Users can opt out of telemetry by setting the environment variable CREWAI_TELEMETRY_OPT_OUT
Fixes:
- #254
- #241
- #372
This branch has been published to https://pypi.org/project/crewai-clean for testing.
As an software architect within a bank which has tons of firewall, such requests for enabling firewall will never get approved , without opting out feature, we will get errors in logs, and we will have a sub standard experience. Unfortunately for us, it would be a no go with crewai if we don't have an opt out of telemetry.
+1 on disabling telemetry for running in airgapped environments, any updates on this?
@joaomdmoura Can you have a look at this one, please? Note in an enterprise environment this is a blocking point
PR Review 🔍
| ⏱️ Estimated effort to review [1-5] |
2, because the PR is small and the changes are straightforward, but it requires careful consideration of the environment variable handling and its impact on the telemetry system. |
| 🧪 Relevant tests |
No |
| ⚡ Possible issues |
Possible Bug: The environment variable check |
| 🔒 Security concerns |
No |
Code feedback:
| relevant file | src/crewai/telemetry/telemetry.py |
| suggestion |
Consider explicitly checking for a specific string value to determine if telemetry should be disabled. This ensures that setting the environment variable to any string will not unintentionally disable telemetry. For example, you could check if the environment variable is set to 'true' or '1'. [important] |
| relevant line | if os.environ.get("CREWAI_TELEMETRY_OPT_OUT", False): |
We are really loving this framework but this feature is important. will this be making it to a release soon. Must have as most of the enterprises will have approve a tool without opt-out.
This is a required option in order to gain enterprise adoption. For the future of crewAI I hope this gets approved by project owners.
This PR is stale because it has been open for 45 days with no activity.
I'm still keen to see this merged.
Now implemented and docs updated :)