runbooks 1.1.2__py3-none-any.whl → 1.1.3__py3-none-any.whl

runbooks/finops/vpc_cleanup_optimizer.py

@@ -1323,7 +1323,7 @@ class VPCCleanupOptimizer:
         monthly_vpc_endpoint_cost = endpoint_result.monthly_cost
         
         # Data processing costs vary by usage - using conservative estimate per region
-        regional_multiplier = pricing_engine.regional_multipliers.get(default_region, 1.0)
+        regional_multiplier = pricing_engine.get_regional_pricing_multiplier('vpc_endpoint', default_region, "us-east-1")
         monthly_data_processing_cost = 50.0 * regional_multiplier  # Base estimate adjusted for region
         
         total_annual_savings = 0.0

runbooks/finops/workspaces_analyzer.py

@@ -75,7 +75,7 @@ class WorkSpacesCostAnalyzer:
     """
     WorkSpaces cost optimization analyzer following enterprise patterns.
     
-    Implements FinOps-24 requirements with proven profile management and Rich CLI standards.
+    Implements WorkSpaces optimization requirements with proven profile management and Rich CLI standards.
     """
     
     def __init__(self, profile: Optional[str] = None):
@@ -84,7 +84,7 @@ class WorkSpacesCostAnalyzer:
         self.profile = get_profile_for_operation("operational", profile)
         self.session = boto3.Session(profile_name=self.profile)
         
-        # FinOps-24 business targets
+        # WorkSpaces optimization business targets
         self.target_annual_savings = 12518.0
         self.unused_threshold_days = 90
         self.analysis_period_days = 30
@@ -286,7 +286,7 @@ class WorkSpacesCostAnalyzer:
         print_header("WorkSpaces Cost Analysis Summary")
         
         summary_table = create_table(
-            title="FinOps-24: WorkSpaces Optimization Summary",
+            title="WorkSpaces Optimization Summary",
             columns=[
                 {"header": "Metric", "style": "cyan"},
                 {"header": "Count", "style": "green bold"},
@@ -555,6 +555,11 @@ def analyze_workspaces(
         Analysis results with cost optimization recommendations
     """
     try:
+        # Initialize variables to prevent scope errors
+        results = []
+        summary = None
+        export_file = None
+
         analyzer = WorkSpacesCostAnalyzer(profile=profile)
         results, summary = analyzer.analyze_workspaces(
             unused_days=unused_days,
@@ -573,19 +578,32 @@ def analyze_workspaces(
             )
         
         # Return comprehensive results
-        return {
-            "summary": summary.to_dict(),
-            "workspaces": [result.to_dict() for result in results],
-            "export_file": export_file,
-            "achievement_rate": summary.target_achievement_rate,
-            "status": "success"
-        }
-        
+        if summary is not None:
+            return {
+                "summary": summary.to_dict(),
+                "workspaces": [result.to_dict() for result in results],
+                "export_file": export_file,
+                "achievement_rate": summary.target_achievement_rate,
+                "status": "success"
+            }
+        else:
+            return {
+                "summary": {"error": "Analysis failed before completion"},
+                "workspaces": [],
+                "export_file": None,
+                "achievement_rate": 0,
+                "status": "partial_failure"
+            }
+
     except Exception as e:
         print_error(f"WorkSpaces analysis failed: {e}")
         return {
             "error": str(e),
-            "status": "failed"
+            "status": "failed",
+            "summary": {"error": str(e)},
+            "workspaces": [],
+            "export_file": None,
+            "achievement_rate": 0
         }