improv(metrics): Adding support to 30 dimensions

Author: leandrodamascena

aws_lambda_powertools/metrics/base.py

@@ -14,7 +14,7 @@
 logger = logging.getLogger(__name__)
 
 MAX_METRICS = 100
-MAX_DIMENSIONS = 9
+MAX_DIMENSIONS = 29
 
 
 class MetricUnit(Enum):
@@ -233,7 +233,7 @@ def add_dimension(self, name: str, value: str) -> None:
             Dimension value
         """
         logger.debug(f"Adding dimension: {name}:{value}")
-        if len(self.dimension_set) == 9:
+        if len(self.dimension_set) == MAX_DIMENSIONS:
             raise SchemaValidationError(
                 f"Maximum number of dimensions exceeded ({MAX_DIMENSIONS}): Unable to add dimension {name}."
             )

tests/functional/test_metrics.py

@@ -320,8 +320,8 @@ def test_schema_no_metrics(service, namespace):
 
 
 def test_exceed_number_of_dimensions(metric, namespace):
-    # GIVEN we have more dimensions than CloudWatch supports
-    dimensions = [{"name": f"test_{i}", "value": "test"} for i in range(11)]
+    # GIVEN we have more dimensions than CloudWatch supports (N+1)
+    dimensions = [{"name": f"test_{i}", "value": "test"} for i in range(31)]
 
     # WHEN we attempt to serialize them into a valid EMF object
     # THEN it should fail validation and raise SchemaValidationError
@@ -332,9 +332,9 @@ def test_exceed_number_of_dimensions(metric, namespace):
 
 
 def test_exceed_number_of_dimensions_with_service(metric, namespace, monkeypatch):
-    # GIVEN we have service set and add more dimensions than CloudWatch supports (N+1)
+    # GIVEN we have service set and add more dimensions than CloudWatch supports (N-1)
     monkeypatch.setenv("POWERTOOLS_SERVICE_NAME", "test_service")
-    dimensions = [{"name": f"test_{i}", "value": "test"} for i in range(9)]
+    dimensions = [{"name": f"test_{i}", "value": "test"} for i in range(29)]
 
     # WHEN we attempt to serialize them into a valid EMF object
     # THEN it should fail validation and raise SchemaValidationError