Explicitly define None as the default value for missing kms_key_id key

in manifest. None should be returned by default, but this makes it
easier to see.
This commit is contained in:
Veli-Matti Lintu 2018-06-14 09:38:36 +03:00
parent 8e24c5e795
commit 2a741a81bf
2 changed files with 2 additions and 2 deletions

View file

@ -26,7 +26,7 @@ def validate_manifest(data, validator, error):
bootloader = data['system']['bootloader']
virtualization = data['provider']['virtualization']
encrypted = data['provider'].get('encrypted', False)
kms_key_id = data['provider'].get('kms_key_id')
kms_key_id = data['provider'].get('kms_key_id', None)
backing = data['volume']['backing']
partition_type = data['volume']['partitions']['type']
enhanced_networking = data['provider']['enhanced_networking'] if 'enhanced_networking' in data['provider'] else None

View file

@ -19,7 +19,7 @@ class Create(Task):
# EBS volumes support encryption. KMS key id is optional and default key
# is used when it is not defined.
encrypted = info.manifest.data['provider'].get('encrypted', False)
kms_key_id = info.manifest.data['provider'].get('kms_key_id')
kms_key_id = info.manifest.data['provider'].get('kms_key_id', None)
info.volume.create(info._ec2['connection'], info._ec2['host']['availabilityZone'], tags=tags, encrypted=encrypted, kms_key_id=kms_key_id)