allow patching on AMIs with different names (with formatting) and lim…

…it only on platform. This allows 2012 machines to update to 2016.
trainline · Jan 24, 2018 · 39300f0 · 39300f0
1 parent 7da4e79
commit 39300f0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/emcli/commands/cycle.py b/emcli/commands/cycle.py
@@ -225,5 +225,5 @@ def validate_ami_compatibility(self, from_name=None, to_name=None):
             if from_name is None:
                 raise ValueError('You must specify --from-ami if --to-ami is given')
         if from_ami is not None and to_ami is not None:
-            if from_ami['AmiType'] != to_ami['AmiType']:
-                raise ValueError('AMI types for from_ami and to_ami must match')
+            if from_ami['Platform'] != to_ami['Platform']:
+                raise ValueError('AMI platforms must match')
diff --git a/emcli/commands/patch.py b/emcli/commands/patch.py
@@ -230,6 +230,6 @@ def validate_ami_compatibility(self, from_name=None, to_name=None):
             if from_name is None:
                 raise ValueError('You must specify --from-ami if --to-ami is given')
         if from_ami is not None and to_ami is not None:
-            if from_ami['AmiType'] != to_ami['AmiType']:
-                raise ValueError('AMI types for from_ami and to_ami must match')
+            if from_ami['Platform'] != to_ami['Platform']:
+                raise ValueError('AMI platforms must match')