🔧 chore(slack): remove unused metrics

[iamrajjoshi]

we don't have any dashboards or use any of these metrics, so lets remove and save us some 💸 it should be close to 1k in savings

also changed some loggers from exception to info since we don't need the stacktrace or issues for a lot of these issues

[codecov]

❌ 5 Tests Failed:

Tests completed	Failed	Passed	Skipped
26365	5	26360	212

View the top 3 failed test(s) by shortest run time

tests.sentry.integrations.slack.test_link_identity.SlackIntegrationLinkIdentityTest::test_basic_flow_with_webhook_client_error

Stack Traces | 2.04s run time

#x1B[1m#x1B[.../integrations/slack/test_link_identity.py#x1B[0m:101: in test_basic_flow_with_webhook_client_error
    assert mock_logger.exception.call_count == 1
#x1B[1m#x1B[31mE   AssertionError: assert 0 == 1#x1B[0m
#x1B[1m#x1B[31mE    +  where 0 = <MagicMock name='_logger.exception' id='139887267145984'>.call_count#x1B[0m
#x1B[1m#x1B[31mE    +    where <MagicMock name='_logger.exception' id='139887267145984'> = <MagicMock name='_logger' id='139887278848192'>.exception#x1B[0m

tests.sentry.integrations.slack.test_link_identity.SlackIntegrationLinkIdentityTest::test_basic_flow_with_web_client_error

Stack Traces | 2.13s run time

#x1B[1m#x1B[.../integrations/slack/test_link_identity.py#x1B[0m:141: in test_basic_flow_with_web_client_error
    assert mock_logger.exception.call_count == 1
#x1B[1m#x1B[31mE   AssertionError: assert 0 == 1#x1B[0m
#x1B[1m#x1B[31mE    +  where 0 = <MagicMock name='_logger.exception' id='140500847472528'>.call_count#x1B[0m
#x1B[1m#x1B[31mE    +    where <MagicMock name='_logger.exception' id='140500847472528'> = <MagicMock name='_logger' id='140500840492608'>.exception#x1B[0m

tests.sentry.incidents.action_handlers.test_slack.SlackActionHandlerTest::test_fire_metric_alert_sdk_error

Stack Traces | 2.77s run time

#x1B[1m#x1B[.../hostedtoolcache/Python/3.13.1................../x64/lib/python3.13/unittest/mock.py#x1B[0m:1421: in patched
    with self.decoration_helper(patched,
#x1B[1m#x1B[.../hostedtoolcache/Python/3.13.1................../x64/lib/python3.13/contextlib.py#x1B[0m:141: in __enter__
    return next(self.gen)
#x1B[1m#x1B[.../hostedtoolcache/Python/3.13.1................../x64/lib/python3.13/unittest/mock.py#x1B[0m:1403: in decoration_helper
    arg = exit_stack.enter_context(patching)
#x1B[1m#x1B[.../hostedtoolcache/Python/3.13.1................../x64/lib/python3.13/contextlib.py#x1B[0m:530: in enter_context
    result = _enter(cm)
#x1B[1m#x1B[.../hostedtoolcache/Python/3.13.1................../x64/lib/python3.13/unittest/mock.py#x1B[0m:1495: in __enter__
    original, local = self.get_original()
#x1B[1m#x1B[.../hostedtoolcache/Python/3.13.1................../x64/lib/python3.13/unittest/mock.py#x1B[0m:1465: in get_original
    raise AttributeError(
#x1B[1m#x1B[31mE   AttributeError: <module 'sentry.integrations.slack.utils.notifications' from '.../slack/utils/notifications.py'> does not have the attribute 'metrics'#x1B[0m

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}

[Christinarlong]

could we add some context/info to help debug the error ?

[Christinarlong]

for here and in the other logger.info -> logger.exception, would it be helpful to add the exc_info or report the error to sentry?