Merge branch 'csm_1_data_model' into csm_2_instrumentation

Author: daniel-sanche

google/cloud/bigtable/data/_async/client.py

@@ -971,9 +971,7 @@ def __init__(
             default_retryable_errors or ()
         )
 
-        self._metrics = BigtableClientSideMetricsController(
-            handlers=[],
-        )
+        self._metrics = BigtableClientSideMetricsController()
 
         try:
             self._register_instance_future = CrossSync.create_task(

google/cloud/bigtable/data/_metrics/metrics_controller.py

@@ -34,9 +34,7 @@ def __init__(
         Initializes the metrics controller.
 
         Args:
-          - interceptor: A metrics interceptor to use for triggering Operation lifecycle events
           - handlers: A list of MetricsHandler objects to subscribe to metrics events.
-          - **kwargs: Optional arguments to pass to the metrics handlers.
         """
         self.handlers: list[MetricsHandler] = handlers or []
 

google/cloud/bigtable/data/_sync_autogen/client.py

@@ -758,7 +758,7 @@ def __init__(
         self.default_retryable_errors: Sequence[type[Exception]] = (
             default_retryable_errors or ()
         )
-        self._metrics = BigtableClientSideMetricsController(handlers=[])
+        self._metrics = BigtableClientSideMetricsController()
         try:
             self._register_instance_future = CrossSync._Sync_Impl.create_task(
                 self.client._register_instance,

tests/unit/data/_metrics/test_metrics_controller.py

@@ -86,4 +86,4 @@ def test_create_operation(self):
         assert op.is_streaming is expected_is_streaming
         assert op.zone is expected_zone
         assert len(op.handlers) == 1
-        assert op.handlers[0] is handler
+        assert op.handlers[0] is handler

tests/unit/data/test_sync_up_to_date.py

@@ -90,7 +90,7 @@ def test_verify_headers(sync_file):
         \#\ distributed\ under\ the\ License\ is\ distributed\ on\ an\ \"AS\ IS\"\ BASIS,\n
         \#\ WITHOUT\ WARRANTIES\ OR\ CONDITIONS\ OF\ ANY\ KIND,\ either\ express\ or\ implied\.\n
         \#\ See\ the\ License\ for\ the\ specific\ language\ governing\ permissions\ and\n
-        \#\ limitations\ under\ the\ License\.
+        \#\ limitations\ under\ the\ License
     """
     pattern = re.compile(license_regex, re.VERBOSE)