chore(slack): track timeout errors from sdk WebClient (#73546)

Author: cathteng

src/sentry/integrations/slack/sdk_client.py

@@ -76,6 +76,13 @@ def wrapped(*args, **kwargs):
             else:
                 logger.info("slack_sdk.missing_error_response", extra={"error": str(e)})
             raise
+        except TimeoutError:
+            metrics.incr(
+                SLACK_DATADOG_METRIC,
+                sample_rate=1.0,
+                tags={"status": "timeout"},
+            )
+            raise
 
         return response
 

tests/sentry/integrations/slack/test_sdk_client.py

@@ -69,3 +69,19 @@ def test_api_call_error(self, mock_api_call, mock_metrics):
             sample_rate=1.0,
             tags={"ok": False, "status": 500},
         )
+
+    @patch("sentry.integrations.slack.sdk_client.metrics")
+    @patch("slack_sdk.web.client.WebClient._perform_urllib_http_request")
+    def test_api_call_timeout_error(self, mock_api_call, mock_metrics):
+        mock_api_call.side_effect = TimeoutError()
+
+        client = SlackSdkClient(integration_id=self.integration.id)
+
+        with pytest.raises(TimeoutError):
+            client.chat_postMessage(channel="#announcements", text="hello")
+
+        mock_metrics.incr.assert_called_with(
+            SLACK_DATADOG_METRIC,
+            sample_rate=1.0,
+            tags={"status": "timeout"},
+        )