cdk-factory 0.18.4__tar.gz → 0.18.5__tar.gz

{cdk_factory-0.18.4 → cdk_factory-0.18.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cdk_factory
-Version: 0.18.4
+Version: 0.18.5
 Summary: CDK Factory. A QuickStarter and best practices setup for CDK projects
 Author-email: Eric Wilson <eric.wilson@geekcafe.com>
 License: MIT License

{cdk_factory-0.18.4 → cdk_factory-0.18.5}/pyproject.toml

@@ -33,7 +33,7 @@ markers = [
 [project]
 
 name = "cdk_factory"
-version = "0.18.4"
+version = "0.18.5"
 authors = [
   { name="Eric Wilson", email="eric.wilson@geekcafe.com" }
 ]

{cdk_factory-0.18.4 → cdk_factory-0.18.5}/src/cdk_factory/interfaces/standardized_ssm_mixin.py

@@ -24,6 +24,8 @@ import os
 import re
 from typing import Dict, Any, Optional, List, Union
 from aws_cdk import aws_ssm as ssm
+from aws_cdk import aws_ec2 as ec2
+from aws_cdk import aws_ecs as ecs
 from constructs import Construct
 from aws_lambda_powertools import Logger
 from cdk_factory.configurations.deployment import DeploymentConfig
@@ -610,3 +612,40 @@ class SsmStandardValidator:
                 errors.append(f"{context}: SSM path should use template variables: {path}")
         
         return errors
+    
+        
+    def parse_subnet_ids_from_ssm(self, subnet_ids_key: str = "subnet_ids") -> List[str]:
+        """
+        Helper function to parse subnet IDs from SSM imports.
+        
+        This common pattern handles:
+        1. Comma-separated subnet ID strings from SSM
+        2. List of subnet IDs from SSM
+        3. CDK Token resolution for deployment-time values
+        
+        Args:
+            subnet_ids_key: The key used for subnet IDs in SSM imports (default: "subnet_ids")
+            
+        Returns:
+            List of subnet IDs (empty list if not found or invalid format)
+        """
+        ssm_imports = self.get_all_ssm_imports()
+        
+        if subnet_ids_key not in ssm_imports:
+            logger.warning(f"No subnet IDs found in SSM imports with key: {subnet_ids_key}")
+            return []
+        
+        subnet_ids = ssm_imports[subnet_ids_key]
+        
+        # Handle comma-separated string or list
+        if isinstance(subnet_ids, str):
+            # Split comma-separated string
+            parsed_ids = [sid.strip() for sid in subnet_ids.split(',') if sid.strip()]
+            logger.info(f"Parsed {len(parsed_ids)} subnet IDs from comma-separated string")
+            return parsed_ids
+        elif isinstance(subnet_ids, list):
+            logger.info(f"Using {len(subnet_ids)} subnet IDs from list")
+            return subnet_ids
+        else:
+            logger.warning(f"Unexpected subnet_ids type: {type(subnet_ids)}")
+            return []

{cdk_factory-0.18.4 → cdk_factory-0.18.5}/src/cdk_factory/stack_library/auto_scaling/auto_scaling_stack_standardized.py

@@ -212,8 +212,20 @@ class AutoScalingStack(IStack, VPCProviderMixin, StandardizedSsmMixin):
         """
         # Primary method: Use standardized SSM imports
         ssm_imports = self._get_ssm_imports()
+        
         if "subnet_ids" in ssm_imports:
-            return ssm_imports["subnet_ids"]
+            subnet_ids = ssm_imports["subnet_ids"]
+            
+            # Handle comma-separated string or list
+            if isinstance(subnet_ids, str):
+                # Split comma-separated string
+                parsed_ids = [sid.strip() for sid in subnet_ids.split(',') if sid.strip()]
+                return parsed_ids
+            elif isinstance(subnet_ids, list):
+                return subnet_ids
+            else:
+                logger.warning(f"Unexpected subnet_ids type: {type(subnet_ids)}")
+                return []
         
         # Fallback: Use VPC provider mixin (backward compatibility)
         elif hasattr(self, '_get_subnets_from_provider'):
@@ -497,18 +509,21 @@ class AutoScalingStack(IStack, VPCProviderMixin, StandardizedSsmMixin):
         # Get the underlying CloudFormation resource to add update policy
         cfn_asg = self.auto_scaling_group.node.default_child
         
-        # Get existing update policy or create new one
-        existing_policy = getattr(cfn_asg, 'update_policy', {})
+        # Get CDK's default policy first (if any)
+        default_policy = getattr(cfn_asg, 'update_policy', {})
         
-        # Merge rolling update policy with existing policy
-        existing_policy["AutoScalingRollingUpdate"] = {
-            "MinInstancesInService": update_policy.get("min_instances_in_service", 1),
-            "MaxBatchSize": update_policy.get("max_batch_size", 1),
-            "PauseTime": f"PT{update_policy.get('pause_time', 300)}S"
+        # Merge with defaults, then use the robust add_override method
+        merged_policy = {
+            **default_policy,  # Preserve CDK defaults
+            "AutoScalingRollingUpdate": {
+                "MinInstancesInService": update_policy.get("min_instances_in_service", 1),
+                "MaxBatchSize": update_policy.get("max_batch_size", 1),
+                "PauseTime": f"PT{update_policy.get('pause_time', 300)}S"
+            }
         }
         
-        # Set the merged policy
-        cfn_asg.update_policy = existing_policy
+        # Use the robust CDK-documented approach
+        cfn_asg.add_override("UpdatePolicy", merged_policy)
         
         logger.info("Added rolling update policy to Auto Scaling Group")
 

cdk_factory-0.18.5/src/cdk_factory/version.py

@@ -0,0 +1 @@
+__version__ = "0.18.5"

cdk_factory-0.18.4/src/cdk_factory/version.py

@@ -1 +0,0 @@
-__version__ = "0.18.4"