Glance patch attached by Dan looks good to me, just having one suggestion for the error message for below scenario;
+ if not len(allowed):
+ LOG.warning(_('Refusing to process VMDK file as '
+ 'vmdk_allowed_types is empty'))
+ raise RuntimeError(_('Invalid VMDK create-type specified'))
instead we should use something like;
raise RuntimeError(_('Image is a VMDK, but no VMDK createType is allowed'))
Glance patch attached by Dan looks good to me, just having one suggestion for the error message for below scenario;
+ if not len(allowed): _('Refusing to process VMDK file as ' _('Invalid VMDK create-type specified'))
+ LOG.warning(
+ 'vmdk_allowed_types is empty'))
+ raise RuntimeError(
instead we should use something like;
raise RuntimeError( _('Image is a VMDK, but no VMDK createType is allowed'))