From fb0dd36dbccf889059ffe98c017049001b674b35 Mon Sep 17 00:00:00 2001 From: Jamil Date: Thu, 10 Jul 2025 14:35:18 -0700 Subject: [PATCH] chore(portal): ignore expected libcluster issue (#9822) Adds another expected error message to the ignore list. We have a different (less noisy) log that will alert us if the cluster is below threshold. --- .../apps/domain/lib/domain/telemetry/sentry.ex | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/elixir/apps/domain/lib/domain/telemetry/sentry.ex b/elixir/apps/domain/lib/domain/telemetry/sentry.ex index baa513c43..d2d621877 100644 --- a/elixir/apps/domain/lib/domain/telemetry/sentry.ex +++ b/elixir/apps/domain/lib/domain/telemetry/sentry.ex @@ -1,16 +1,20 @@ defmodule Domain.Telemetry.Sentry do + @ignored_message_patterns [ + # This happens when libcluster loses connection to a node, which is normal during deploys. + # We have threshold-based error logging in Domain.Cluster.GoogleComputeLabelsStrategy to report those. + "Node ~p not responding **~n** Removing (timedout) connection", + "[libcluster:default] unable to connect to" + ] + def before_send(%{original_exception: %{skip_sentry: skip_sentry}}) when skip_sentry do nil end def before_send(%{message: %{formatted: formatted_message}} = event) when is_binary(formatted_message) do - if String.contains?( - formatted_message, - "Node ~p not responding **~n** Removing (timedout) connection" - ) do - # This happens when libcluster loses connection to a node, which is normal during deploys. - # We have threshold-based error logging in Domain.Cluster.GoogleComputeLabelsStrategy to report those. + if Enum.any?(@ignored_message_patterns, fn p -> + String.contains?(formatted_message, p) + end) do nil else event