From 1198123d78f29dbc20b7eda55ef401d873a0578e Mon Sep 17 00:00:00 2001 From: Aseem Bansal Date: Tue, 15 Mar 2022 00:55:22 +0530 Subject: [PATCH] fix: telemetry failure should not cause CLI failure (#4406) --- .../src/datahub/telemetry/telemetry.py | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/metadata-ingestion/src/datahub/telemetry/telemetry.py b/metadata-ingestion/src/datahub/telemetry/telemetry.py index 5e276d61c0..bd98f576d7 100644 --- a/metadata-ingestion/src/datahub/telemetry/telemetry.py +++ b/metadata-ingestion/src/datahub/telemetry/telemetry.py @@ -123,14 +123,17 @@ class Telemetry: return logger.info("Sending init Telemetry") - self.mp.people_set( - self.client_id, - { - "datahub_version": datahub_package.nice_version_name(), - "os": platform.system(), - "python_version": platform.python_version(), - }, - ) + try: + self.mp.people_set( + self.client_id, + { + "datahub_version": datahub_package.nice_version_name(), + "os": platform.system(), + "python_version": platform.python_version(), + }, + ) + except Exception as e: + logger.debug(f"Error reporting telemetry: {e}") self.init_track = True def ping(