Added export of connection advanced metrics

[vladvildanov]

Description of change

Please provide a description of the change here.

Pull Request check-list

Please make sure to review and check all of these items:

• Do tests and lints pass with this change?
• Do the CI tests pass with this change (enable it first in your forked repo and wait for the github action build to finish)?
• Is the new or changed code fully tested?
• Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?
• Is there an example added to the examples folder (if applicable)?

NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.

[Copilot]

Pull request overview

This pull request adds support for exporting connection advanced metrics in the Redis Python client's observability system. The PR introduces structured close reasons via a CloseReason enum and adds new connection lifecycle events for tracking connection acquisition and closure.

Changes:

• Introduces CloseReason enum with three values: APPLICATION_CLOSE, ERROR, and HEALTHCHECK_FAILED to provide structured close reason tracking
• Adds AfterConnectionAcquiredEvent and AfterConnectionClosedEvent to the event system for tracking connection lifecycle
• Modifies record_connection_closed to use the CloseReason enum instead of string-based reasons, removing the pool_name parameter
• Updates default metric groups to exclude COMMAND metrics (now CONNECTION_BASIC | RESILIENCY)
• Adds comprehensive test coverage for new connection events

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
redis/observability/metrics.py	Introduces CloseReason enum and updates record_connection_closed to use it, removing pool_name parameter from the method
redis/observability/recorder.py	Updates record_connection_closed signature to accept CloseReason enum instead of string, removes pool_name parameter
redis/event.py	Adds AfterConnectionAcquiredEvent, AfterConnectionClosedEvent, and corresponding metric export listeners
redis/connection.py	Dispatches new connection events during connect, disconnect, and error scenarios with appropriate CloseReason values
tests/test_observability/test_recorder.py	Updates tests to use CloseReason enum, removes pool_name from test calls
tests/test_observability/test_config.py	Updates default metric groups test to reflect removal of COMMAND from defaults
tests/test_connection_pool.py	Adds comprehensive test coverage for new connection acquired, released, closed, and error events

Comments suppressed due to low confidence (1)

redis/observability/metrics.py:433

• The pool_name parameter has been removed from record_connection_closed, which means the metric will no longer include the pool name attribute. This significantly reduces the observability value of this metric since users won't be able to identify which connection pool had connections closed. This is especially important for applications with multiple connection pools.

    def record_connection_closed(
            self,
            close_reason: Optional[CloseReason] = None,
            error_type: Optional[Exception] = None,
    ) -> None:
        """
        Record a connection closed event.

        Args:
            close_reason: Reason for closing (e.g., 'error', 'application_close')
            error_type: Error type if closed due to error
        """
        if not hasattr(self, "connection_closed"):
            return

        attrs = self.attr_builder.build_connection_attributes()
        if close_reason:
            attrs[REDIS_CLIENT_CONNECTION_CLOSE_REASON] = close_reason.value

        attrs.update(
            self.attr_builder.build_error_attributes(
                error_type=error_type,
            )
        )

        self.connection_closed.add(1, attributes=attrs)

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[petyaslavova]

LGTM.