Size/IOPS should be positive_integers (cloudtools#761)

phobologic · markpeek · commit 7ec39ccc23ad · 2017-07-09T12:36:53.000-07:00
This isn't 100% correct, only because positive_integers allows for
a value of 0, but changing that functionality seems like it'd break
things elsewhere.
diff --git a/troposphere/ec2.py b/troposphere/ec2.py
@@ -464,9 +464,9 @@ class Volume(AWSObject):
         'AutoEnableIO': (boolean, False),
         'AvailabilityZone': (basestring, True),
         'Encrypted': (boolean, False),
-        'Iops': (integer, False),
+        'Iops': (positive_integer, False),
         'KmsKeyId': (basestring, False),
-        'Size': (basestring, False),
+        'Size': (positive_integer, False),
         'SnapshotId': (basestring, False),
         'Tags': (list, False),
         'VolumeType': (basestring, False),
