diff --git a/src/sentry/integrations/slack/service.py b/src/sentry/integrations/slack/service.py index 9ca8c60e8ff22e..b857390a0a9fc6 100644 --- a/src/sentry/integrations/slack/service.py +++ b/src/sentry/integrations/slack/service.py @@ -141,7 +141,7 @@ def notify_all_threads_for_activity(self, activity: Activity) -> None: } if activity.group is None: - self._logger.info( + self._logger.debug( "no group associated on the activity, nothing to do", extra=log_params, ) @@ -156,7 +156,8 @@ def notify_all_threads_for_activity(self, activity: Activity) -> None: ) if activity.user_id is None and not uptime_resolved_notification: - self._logger.info( + # This is a machine/system update, so we don't need to send a notification + self._logger.debug( "machine/system updates are ignored at this time, nothing to do", extra=log_params, ) diff --git a/tests/sentry/integrations/slack/service/test_slack_service.py b/tests/sentry/integrations/slack/service/test_slack_service.py index 0750dabf53c650..b6781d14decb5a 100644 --- a/tests/sentry/integrations/slack/service/test_slack_service.py +++ b/tests/sentry/integrations/slack/service/test_slack_service.py @@ -128,7 +128,7 @@ def test_none_group(self): with mock.patch.object(self.service, "_logger") as mock_logger: self.service.notify_all_threads_for_activity(activity=self.activity) - mock_logger.info.assert_called_with( + mock_logger.debug.assert_called_with( "no group associated on the activity, nothing to do", extra={ "activity_id": self.activity.id, @@ -141,7 +141,7 @@ def test_none_user_id(self): with mock.patch.object(self.service, "_logger") as mock_logger: self.service.notify_all_threads_for_activity(activity=self.activity) - mock_logger.info.assert_called_with( + mock_logger.debug.assert_called_with( "machine/system updates are ignored at this time, nothing to do", extra={ "activity_id": self.activity.id,