Cleaner log messages from ecs_composex.compose.compose_services.Compo…

…seService.ephemeral_storage() compose-x#719
thorfi · Nov 29, 2023 · 132feb9 · 132feb9
1 parent ea734af
commit 132feb9
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/ecs_composex/compose/compose_services/__init__.py b/ecs_composex/compose/compose_services/__init__.py
@@ -504,10 +504,8 @@ def container_start_condition(self, value):
     @property
     def ephemeral_storage(self):
         storage_key = "ecs.ephemeral.storage"
-        storage_value = set_else_none(storage_key, self.deploy_labels)
-        if storage_value is None:
-            return 0
-        elif isinstance(storage_value, (int, float)):
+        storage_value = set_else_none(storage_key, self.deploy_labels, 0)
+        if isinstance(storage_value, (int, float)):
             ephemeral_storage = int(storage_value)
         elif isinstance(storage_value, str):
             ephemeral_storage = int(set_memory_to_mb(storage_value) / 1024)
@@ -518,12 +516,13 @@ def ephemeral_storage(self):
                 "Expected one of",
                 [int, float, str],
             )
-        LOG.info(f"{self.name} - {storage_key} set to {ephemeral_storage}")
         if ephemeral_storage <= 21:
             return 0
         elif ephemeral_storage > 200:
+            LOG.warning(f"{self.name} - {storage_key} set to maximum 200 ({ephemeral_storage} > 200)")
             return 200
         else:
+            LOG.info(f"{self.name} - {storage_key} set to {ephemeral_storage}")
             return int(ephemeral_storage)
 
     @property